Skip to content
This repository was archived by the owner on Feb 22, 2023. It is now read-only.

Conversation

@ditman
Copy link
Member

@ditman ditman commented Feb 27, 2020

Reverts #2560

@ditman ditman requested a review from collinjackson February 27, 2020 01:11
@ditman ditman requested a review from amirh February 27, 2020 01:11
Copy link
Contributor

@collinjackson collinjackson left a 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.

@ditman
Copy link
Member Author

ditman commented Feb 27, 2020

(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)
@jonasfj
Copy link
Member

jonasfj commented Feb 27, 2020

2.0.0 should now be removed from pub.dev

@ditman
Copy link
Member Author

ditman commented Feb 27, 2020

This change will always have a red pre-submit, since the version script checks against master and not what's published in pub.dev. Post-submit, the repo should go green again.

@collinjackson
Copy link
Contributor

OK, sounds good.

No need to publish a patch bump in this case, this'll land on red regardless.
No need for the patch bump.
@ditman
Copy link
Member Author

ditman commented Feb 27, 2020

Merging on red, since most of the failures are related to versioning, which is what we're trying to straighten out here.

Running version check for changed packages
packages/connectivity/connectivity_platform_interface/pubspec.yaml incorrectly updated version.
HEAD: 1.0.3, master: 2.0.0.
Allowed versions: {3.0.0: NextVersionType.BREAKING_MAJOR, 2.1.0: NextVersionType.MINOR, 2.0.1: NextVersionType.PATCH}

@ditman ditman merged commit 73033b4 into master Feb 27, 2020
@ditman ditman deleted the revert-2560-connectivity-unknown-platform branch February 27, 2020 19:29
EdwinRomelta pushed a commit to EdwinRomelta/plugins that referenced this pull request Jun 11, 2020
…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.
FlutterSu pushed a commit to FlutterSu/flutter-plugins that referenced this pull request Nov 20, 2020
…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.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants