Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Switch to http_image_provider 1.0 #1391

Merged
merged 1 commit into from
Oct 28, 2024
Merged

Switch to http_image_provider 1.0 #1391

merged 1 commit into from
Oct 28, 2024

Conversation

brianquinlan
Copy link
Collaborator

  • I’ve reviewed the contributor guide and applied the relevant portions to this PR.
Contribution guidelines:

Note that many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.

@github-actions github-actions bot added package:cupertino_http Issues related to package:cupertino_http package:cronet_http labels Oct 24, 2024
@github-actions GitHub Actions
Copy link

Package publishing

Package Version Status Publish tag (post-merge)
package:http2 2.3.1 ready to publish http2-v2.3.1
package:cronet_http 1.3.3-wip WIP (no publish necessary)
package:http 1.2.3-wip WIP (no publish necessary)
package:http_parser 4.1.1 already published at pub.dev
package:http_profile 0.1.1-wip WIP (no publish necessary)
package:ok_http 0.1.1-wip WIP (no publish necessary)
package:web_socket 0.1.6 already published at pub.dev
package:cupertino_http 1.5.2-wip (error) pubspec version (1.5.2-wip) and changelog (2.0.0-wip) don't agree

Documentation at https://github.com/dart-lang/ecosystem/wiki/Publishing-automation.

@github-actions GitHub Actions
Copy link

PR Health

Changelog Entry ❗
Package Changed Files
package:cronet_http pkgs/cronet_http/example/pubspec.yaml
package:cupertino_http pkgs/cupertino_http/example/pubspec.yaml
package:ok_http pkgs/ok_http/example/pubspec.yaml

Changes to files need to be accounted for in their respective changelogs.

This check can be disabled by tagging the PR with skip-changelog-check.

Coverage ✔️
File Coverage

This check for test coverage is informational (issues shown here will not fail the PR).

License Headers ✔️ 
// Copyright (c) 2024, the Dart project authors. Please see the AUTHORS file
// for details. All rights reserved. Use of this source code is governed by a
// BSD-style license that can be found in the LICENSE file.
Files
no missing headers

All source files should start with a license header.

Unrelated files missing license headers
Files
pkgs/cronet_http/lib/src/jni/jni_bindings.dart
pkgs/cupertino_http/lib/src/native_cupertino_bindings.dart
pkgs/http/example/main.dart
pkgs/http2/test/src/flowcontrol/mocks.mocks.dart
pkgs/http_client_conformance_tests/example/client_test.dart
pkgs/http_client_conformance_tests/lib/src/compressed_response_body_server_vm.dart
pkgs/http_client_conformance_tests/lib/src/compressed_response_body_server_web.dart
pkgs/http_client_conformance_tests/lib/src/dummy_isolate.dart
pkgs/http_client_conformance_tests/lib/src/multipart_server_vm.dart
pkgs/http_client_conformance_tests/lib/src/multipart_server_web.dart
pkgs/http_client_conformance_tests/lib/src/multiple_clients_server_vm.dart
pkgs/http_client_conformance_tests/lib/src/multiple_clients_server_web.dart
pkgs/http_client_conformance_tests/lib/src/redirect_server_vm.dart
pkgs/http_client_conformance_tests/lib/src/redirect_server_web.dart
pkgs/http_client_conformance_tests/lib/src/request_body_server_vm.dart
pkgs/http_client_conformance_tests/lib/src/request_body_server_web.dart
pkgs/http_client_conformance_tests/lib/src/request_body_streamed_server_vm.dart
pkgs/http_client_conformance_tests/lib/src/request_body_streamed_server_web.dart
pkgs/http_client_conformance_tests/lib/src/request_cookies_server_vm.dart
pkgs/http_client_conformance_tests/lib/src/request_cookies_server_web.dart
pkgs/http_client_conformance_tests/lib/src/request_headers_server_vm.dart
pkgs/http_client_conformance_tests/lib/src/request_headers_server_web.dart
pkgs/http_client_conformance_tests/lib/src/request_methods_server_vm.dart
pkgs/http_client_conformance_tests/lib/src/request_methods_server_web.dart
pkgs/http_client_conformance_tests/lib/src/response_body_server_vm.dart
pkgs/http_client_conformance_tests/lib/src/response_body_server_web.dart
pkgs/http_client_conformance_tests/lib/src/response_body_streamed_server_vm.dart
pkgs/http_client_conformance_tests/lib/src/response_body_streamed_server_web.dart
pkgs/http_client_conformance_tests/lib/src/response_cookies_server_vm.dart
pkgs/http_client_conformance_tests/lib/src/response_cookies_server_web.dart
pkgs/http_client_conformance_tests/lib/src/response_headers_server_vm.dart
pkgs/http_client_conformance_tests/lib/src/response_headers_server_web.dart
pkgs/http_client_conformance_tests/lib/src/response_status_line_server_vm.dart
pkgs/http_client_conformance_tests/lib/src/response_status_line_server_web.dart
pkgs/http_client_conformance_tests/lib/src/server_errors_server_vm.dart
pkgs/http_client_conformance_tests/lib/src/server_errors_server_web.dart
pkgs/http_parser/test/example_test.dart
pkgs/http_profile/lib/http_profile.dart
pkgs/java_http/lib/src/third_party/java/io/BufferedInputStream.dart
pkgs/java_http/lib/src/third_party/java/io/InputStream.dart
pkgs/java_http/lib/src/third_party/java/io/OutputStream.dart
pkgs/java_http/lib/src/third_party/java/io/_package.dart
pkgs/java_http/lib/src/third_party/java/lang/System.dart
pkgs/java_http/lib/src/third_party/java/lang/_package.dart
pkgs/java_http/lib/src/third_party/java/net/HttpURLConnection.dart
pkgs/java_http/lib/src/third_party/java/net/URL.dart
pkgs/java_http/lib/src/third_party/java/net/URLConnection.dart
pkgs/java_http/lib/src/third_party/java/net/_package.dart
pkgs/ok_http/lib/src/jni/bindings.dart
pkgs/web_socket/example/web_socket_example.dart
pkgs/web_socket/lib/testing.dart
pkgs/web_socket_conformance_tests/example/client_test.dart
pkgs/web_socket_conformance_tests/lib/src/close_local_server_vm.dart
pkgs/web_socket_conformance_tests/lib/src/close_local_server_web.dart
pkgs/web_socket_conformance_tests/lib/src/close_remote_server_vm.dart
pkgs/web_socket_conformance_tests/lib/src/close_remote_server_web.dart
pkgs/web_socket_conformance_tests/lib/src/continuously_writing_server_vm.dart
pkgs/web_socket_conformance_tests/lib/src/continuously_writing_server_web.dart
pkgs/web_socket_conformance_tests/lib/src/disconnect_after_upgrade_server_vm.dart
pkgs/web_socket_conformance_tests/lib/src/disconnect_after_upgrade_server_web.dart
pkgs/web_socket_conformance_tests/lib/src/echo_server_vm.dart
pkgs/web_socket_conformance_tests/lib/src/echo_server_web.dart
pkgs/web_socket_conformance_tests/lib/src/no_upgrade_server_vm.dart
pkgs/web_socket_conformance_tests/lib/src/no_upgrade_server_web.dart
pkgs/web_socket_conformance_tests/lib/src/peer_protocol_errors_server_vm.dart
pkgs/web_socket_conformance_tests/lib/src/peer_protocol_errors_server_web.dart
pkgs/web_socket_conformance_tests/lib/src/protocol_server_vm.dart
pkgs/web_socket_conformance_tests/lib/src/protocol_server_web.dart

@brianquinlan brianquinlan merged commit 99475f2 into dart-lang:master Oct 28, 2024
32 of 37 checks passed
copybara-service bot pushed a commit to dart-lang/sdk that referenced this pull request Oct 31, 2024
@devoncarew
[deps] rev bazel_worker, benchmark_harness, browser_launcher, cli_uti…
ae0510d 
…l, csslib, ecosystem, glob, html, http, http_multi_server, json_rpc_2, markdown, mockito, pub_semver, source_span, sse, stack_trace, string_scanner, term_glyph, test, test_descriptor, test_process, vector_math, web_socket_channel, yaml

Revisions updated by `dart tools/rev_sdk_deps.dart`.

bazel_worker (https://github.com/dart-lang/bazel_worker/compare/aa3cc9e..7d36032):
  7d36032  2024-10-29  Moritz  Update README.md before archiving (dart-archive/bazel_worker#97)
  ffdecd3  2024-10-29  Kevin Moore  blast repo changes: drop-lint (dart-archive/bazel_worker#96)

benchmark_harness (https://github.com/dart-lang/benchmark_harness/compare/44f125a..ccdb583):
  ccdb583  2024-10-31  Moritz  Update README.md before archiving (dart-archive/benchmark_harness#112)

browser_launcher (https://github.com/dart-lang/browser_launcher/compare/e5fc5d4..f9b48d2):
  f9b48d2  2024-10-28  Moritz  Update README.md before archiving (dart-archive/browser_launcher#66)

cli_util (https://github.com/dart-lang/cli_util/compare/c36b394..6a61e53):
  6a61e53  2024-10-31  Moritz  Update README.md before archiving (dart-archive/cli_util#108)

csslib (https://github.com/dart-lang/csslib/compare/a3700b0..7f5f84e):
  7f5f84e  2024-10-31  Moritz  Update README.md before archiving (dart-archive/csslib#208)

ecosystem (https://github.com/dart-lang/ecosystem/compare/66ddc4f..5099841):
  5099841  2024-10-31  Moritz  Update instructions for trebuchet (dart-lang/ecosystem#306)
  8300570  2024-10-28  Devon Carew  fix the labeller config file (dart-lang/ecosystem#316)
  5a1f899  2024-10-28  Kevin Moore  new tweak: drop lint (dart-lang/ecosystem#315)
  50dd246  2024-10-28  Kevin Moore  trebuchet: drop unneeded lints dep (dart-lang/ecosystem#314)

glob (https://github.com/dart-lang/glob/compare/00a9c82..3e414a2):
  3e414a2  2024-10-28  Kevin Moore  blast_repo fixes (dart-lang/glob#98)

html (https://github.com/dart-lang/html/compare/6d3bc86..929b6de):
  929b6de  2024-10-31  Moritz  Update README.md (dart-archive/html#258)

http (https://github.com/dart-lang/http/compare/f5ec11c..8db0d0a):
  8db0d0a  2024-10-30  Kevin Moore  Drop outdated lint (dart-lang/http#1395)
  71d5212  2024-10-28  Brian Quinlan  Prepare to publish cupertino_http 2.0.0 (dart-lang/http#1390)
  99475f2  2024-10-28  Brian Quinlan  Switch to http_image_provider 1.0 (dart-lang/http#1391)

http_multi_server (https://github.com/dart-lang/http_multi_server/compare/e7515b5..a9e71fa):
  a9e71fa  2024-10-28  Kevin Moore  blast_repo fixes (dart-lang/http_multi_server#73)

json_rpc_2 (https://github.com/dart-lang/json_rpc_2/compare/c9b616b..cd94cef):
  cd94cef  2024-10-28  Moritz  Update README.md before archiving (dart-archive/json_rpc_2#119)

markdown (https://github.com/dart-lang/markdown/compare/207bb44..a4b1d2c):
  a4b1d2c  2024-10-30  Mack Myers  Adds export for link_reference_definition_syntax.dart (dart-lang/markdown#626)
  24412ee  2024-10-29  Kevin Moore  blast_repo fixes (dart-lang/markdown#627)

mockito (https://github.com/dart-lang/mockito/compare/57d484f..f72791d):
  f72791d  2024-10-31  Bob Nystrom  In my previous change, I tried a simpler approach in custom_mocks_test to try to not be sensitive to formatting changes before I realized that wouldn't work in auto_mocks_test and came up with something more robust.
  0d582b7  2024-10-30  Bob Nystrom  In custom_mocks_test, there were only a handful that were sensitive to formatting, so I just loosened the tests a bit manually.

pub_semver (https://github.com/dart-lang/pub_semver/compare/8cce9d0..72317ea):
  72317ea  2024-10-28  Kevin Moore  blast_repo fixes (dart-lang/pub_semver#108)

source_span (https://github.com/dart-lang/source_span/compare/ec100b7..e518512):
  e518512  2024-10-28  Kevin Moore  blast_repo fixes (dart-lang/source_span#118)

sse (https://github.com/dart-lang/sse/compare/af2c5c5..1b02011):
  1b02011  2024-10-28  dependabot[bot]  Bump actions/checkout from 4.1.7 to 4.2.0 in the github-actions group (dart-lang/sse#116)
  e4ab625  2024-10-28  Kevin Moore  blast_repo fixes (dart-lang/sse#117)

stack_trace (https://github.com/dart-lang/stack_trace/compare/115bcd9..582891c):
  582891c  2024-10-28  Kevin Moore  blast_repo fixes (dart-lang/stack_trace#164)

string_scanner (https://github.com/dart-lang/string_scanner/compare/084b201..4de83f0):
  4de83f0  2024-10-28  Kevin Moore  blast_repo fixes (dart-lang/string_scanner#83)

term_glyph (https://github.com/dart-lang/term_glyph/compare/19d8c08..d7d8d7c):
  d7d8d7c  2024-10-29  Kevin Moore  blast_repo fixes (dart-lang/term_glyph#57)

test (https://github.com/dart-lang/test/compare/fe38650..42495c2):
  42495c26  2024-10-29  Nate Bosch  Enable avoid_dynamic_calls lint (dart-lang/test#2410)
  9a267def  2024-10-29  Nate Bosch  Remove some unused testing utilities (dart-lang/test#2413)

test_descriptor (https://github.com/dart-lang/test_descriptor/compare/a3db1ef..aa99e99):
  aa99e99  2024-10-29  Kevin Moore  blast_repo fixes (dart-lang/test_descriptor#72)

test_process (https://github.com/dart-lang/test_process/compare/52ee3f5..c4986dd):
  c4986dd  2024-10-29  Kevin Moore  blast_repo fixes (dart-lang/test_process#63)

vector_math (https://github.com/google/vector_math.dart/compare/da9889f..3937447):
  3937447  2024-10-28  Kevin Moore  blast_repo fixes (google/vector_math.dart#333)

web_socket_channel (https://github.com/dart-lang/web_socket_channel/compare/40aa29f..1f15eca):
  1f15eca  2024-10-29  Kevin Moore  blast_repo fixes (dart-lang/web_socket_channel#385)

yaml (https://github.com/dart-lang/yaml/compare/e773005..6cc2745):
  6cc2745  2024-10-28  Kevin Moore  blast_repo fixes (dart-lang/yaml#171)

Change-Id: I0b3e0e1f9195bf4c1296b38a3b8839337444fd6d
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/393040
Reviewed-by: Konstantin Shcheglov <[email protected]>
Commit-Queue: Devon Carew <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@liamappelbe liamappelbe liamappelbe approved these changes

@natebosch natebosch Awaiting requested review from natebosch

Assignees
No one assigned
Labels
package:cronet_http package:cupertino_http Issues related to package:cupertino_http
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@brianquinlan @liamappelbe