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

Revert AndroidX changes for 1.0.0 plugins #1127

Merged
merged 2 commits into from
Jan 28, 2019
Merged

Revert AndroidX changes for 1.0.0 plugins #1127

merged 2 commits into from
Jan 28, 2019

Conversation

mklim
Copy link
Contributor

@mklim mklim commented Jan 28, 2019

The AndroidX migration didn't increment the major version for plugins >=
1.0.0. This reverts it in a patch, forward roll with a major update to follow in
a separate PR.

It's critical that the forward roll PR is published with this so that clients
that have previously migrated aren't broken by the revert. I'm not planning
on merging this into master until the forward roll (#1128) is
also LGTMed and ready to merge.

The AndroidX migration didn't increment the major version for plugins >=
1.0.0. This commit reverts the code changes in the affected plugins.
Copy link
Contributor

@dnfield dnfield left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@amirh amirh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mklim mklim merged commit ea08bab into flutter:master Jan 28, 2019
@mklim mklim deleted the revert_androidx branch January 28, 2019 22:28
andreidiaconu pushed a commit to andreidiaconu/plugins that referenced this pull request Feb 17, 2019
The AndroidX migration didn't increment the major version for plugins >=
1.0.0. This commit reverts the code changes in the affected plugins.

It's critical that the forward roll PR is published with this so that clients
that have previously migrated aren't broken by the revert. I'm not planning
on merging this into master until the forward roll (flutter#1128) is
also LGTMed and ready to merge.

flutter/flutter#27128
flutter/flutter#23995
andreidiaconu added a commit to andreidiaconu/plugins that referenced this pull request Feb 17, 2019
julianscheel pushed a commit to jusst-engineering/plugins that referenced this pull request Mar 11, 2020
The AndroidX migration didn't increment the major version for plugins >=
1.0.0. This commit reverts the code changes in the affected plugins.

It's critical that the forward roll PR is published with this so that clients
that have previously migrated aren't broken by the revert. I'm not planning
on merging this into master until the forward roll (flutter#1128) is
also LGTMed and ready to merge.

flutter/flutter#27128
flutter/flutter#23995
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.

4 participants