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

Revert "Upgrade Pex to 2.1.95 and default version of black to 22.6.0." #16128

Merged
merged 1 commit into from
Jul 11, 2022

Conversation

benjyw
Copy link
Contributor

@benjyw benjyw commented Jul 11, 2022

This reverts #16116, as that changed caused wheel building to fail.

To observe this failure, run ./build-support/bin/release.sh build-wheels

[ci skip-rust]

[ci skip-build-wheels]

@benjyw benjyw changed the title Revert "Upgrade Pex to 2.1.95. (#16110)" (#16125) Revert "Upgrade Pex to 2.1.95. (cherry-pick of #16110)" (#16125) Jul 11, 2022
@benjyw benjyw requested a review from jsirois July 11, 2022 15:46
group = "org.junit.platform"
artifact = "junit-platform-commons"
version = "1.7.2"
group = "org.apiguardian"
Copy link
Contributor

Choose a reason for hiding this comment

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

Hrm, I missed this in the main review. There is no --python-only flag to the re-gen script; so I had just reverted all JVM lockfile updates when I did the initial upgrade on main to 2.1.95. It's probably fine though / tests will cover the lock update as they always do or don't.

@benjyw benjyw force-pushed the cherrypick_16125_to_2.13.x branch from 8aeb941 to c0c950c Compare July 11, 2022 18:51
… (Cherry-picks of pantsbuild#16110 & pantsbuild#15960) (pantsbuild#16116)"

This reverts commit 0987300.

[ci skip-rust]

[ci skip-build-wheels]
@benjyw benjyw force-pushed the cherrypick_16125_to_2.13.x branch from c0c950c to 5f31569 Compare July 11, 2022 19:18
@benjyw benjyw changed the title Revert "Upgrade Pex to 2.1.95. (cherry-pick of #16110)" (#16125) Revert "Upgrade Pex to 2.1.95 and default version of black to 22.6.0." Jul 11, 2022
@benjyw
Copy link
Contributor Author

benjyw commented Jul 11, 2022

Ended up doing this as a revert of the cherrypick of the upgrade, instead of a cherrypick of the revert in main, since the latter was very unclean and caused many tests to fail.

@benjyw benjyw merged commit 657bd9c into pantsbuild:2.13.x Jul 11, 2022
@benjyw benjyw deleted the cherrypick_16125_to_2.13.x branch July 11, 2022 19:42
@stuhood stuhood mentioned this pull request Jul 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants