close
The Wayback Machine - https://web.archive.org/web/20210908155246/https://github.com/flutter/plugins/pull/3723
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.

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

[integration_test] Deprecate, and stop using in this repository #3723

Merged

Conversation

@stuartmorgan
Copy link
Contributor

@stuartmorgan stuartmorgan commented Mar 17, 2021

  • Updates integration_test README to note that the published version is deprecated and direct people to the SDK version.
  • Updates all plugins here to use the SDK version.

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read and followed the Flutter Style Guide and the C++, Objective-C, Java style guides.
  • I signed the CLA.
  • The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
  • I listed at least one issue that this PR fixes in the description above.
  • I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
  • I updated CHANGELOG.md to add a description of the change.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
  • All existing and new tests are passing.
@stuartmorgan
Copy link
Contributor Author

@stuartmorgan stuartmorgan commented Mar 17, 2021

This doesn't update the version and CHANGELOG of all the plugins it touches since I would not expect this to be a change that matters in the published version, thus shouldn't need publishing. If there's a reason we need to publish that I missed, I can update the PR accordingly.

@ditman
Copy link
Member

@ditman ditman commented Mar 19, 2021

At some point, we need to ask people to unpublish this, maybe? https://pub.dev/packages/integration_test Or mark it as "deprecated"?

@stuartmorgan
Copy link
Contributor Author

@stuartmorgan stuartmorgan commented Mar 19, 2021

Yes, I was going to publish this now to guide people via the README, then eventually (I need to think about how long to wait) mark it as discontinued.

@ditman
Copy link
Member

@ditman ditman commented Mar 19, 2021

@stuartmorgan I've seen that pub.dev redirects to the flutter API docs for some packages. I'm not sure how that's achieved, but check out this link: http://pub.dev/packages/flutter_test. It'd be nice that integration_test eventually redirected to api.flutter.dev as well!

@stuartmorgan
Copy link
Contributor Author

@stuartmorgan stuartmorgan commented Mar 19, 2021

Oh, that's nice! @jonasfj what's the mechanism for adding a redirect like that?

engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 19, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 19, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 19, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 20, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 22, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 23, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 23, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 24, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 24, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment