-
Notifications
You must be signed in to change notification settings - Fork 9.7k
Revert "[connectivity_platform_interface] Add ConnectivityResult.unknown."
#2561
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks for giving us some time to figure out the right approach here.
|
(I added a few changes to roll this "back" to 1.0.3+1 (instead of 1.0.3) so this PR can be checked to be fully green when 2.0.0 is removed from pub.dev, otherwise the checks will complain that 1.0.3 is already published after 2.0.0 is gone) |
Change to 1.0.3+1 so this can be merged easily after 2.0.0 is removed.
Document why 1.0.3+1 version was needed (to remove version 2.0.0)
|
|
|
This change will always have a red pre-submit, since the version script checks against |
|
OK, sounds good. |
packages/connectivity/connectivity_platform_interface/CHANGELOG.md
Outdated
Show resolved
Hide resolved
No need to publish a patch bump in this case, this'll land on red regardless.
No need for the patch bump.
|
Merging on red, since most of the failures are related to versioning, which is what we're trying to straighten out here. |
…nown`." (flutter#2561) The previous push introduced a new major version into the connectivity_platform_interface package. That might not be the best approach. The new version has been unpublished from pub.dev, and this change restores the repo to its previous state. This reverts commit 4411247.
…nown`." (flutter#2561) The previous push introduced a new major version into the connectivity_platform_interface package. That might not be the best approach. The new version has been unpublished from pub.dev, and this change restores the repo to its previous state. This reverts commit 4411247.
Reverts #2560