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

Remove more 2.18.0.dev0 deprecations (follow up to #19073) #19077

Merged
merged 2 commits into from
May 21, 2023

Conversation

cognifloyd
Copy link
Member

Remove the export_virtualenv_for_targets rule which is unused as of #19073 where we removed support for calling the export goal without --resolve.

Copy link
Sponsor Contributor

@benjyw benjyw left a comment

Choose a reason for hiding this comment

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

Good catch!

@cognifloyd cognifloyd enabled auto-merge (squash) May 21, 2023 17:04
@cognifloyd
Copy link
Member Author

I'm not sure what's up with the CI failure that said to regenerate CI config. Why would this PR have anything to do with that?

@benjyw
Copy link
Sponsor Contributor

benjyw commented May 21, 2023

Strange. What do you get if you regen?

@benjyw
Copy link
Sponsor Contributor

benjyw commented May 21, 2023

I'm seeing the same thing in #19079

@benjyw
Copy link
Sponsor Contributor

benjyw commented May 21, 2023

That was a bad merge on #18829, will fix

@benjyw
Copy link
Sponsor Contributor

benjyw commented May 21, 2023

Fix in #19080

@cognifloyd cognifloyd disabled auto-merge May 21, 2023 20:35
The export_virtualenv_for_targets rule is unused as of pantsbuild#19073
@cognifloyd cognifloyd enabled auto-merge (squash) May 21, 2023 20:36
@cognifloyd cognifloyd merged commit f410994 into pantsbuild:main May 21, 2023
@cognifloyd cognifloyd deleted the more-2.18-deprecations branch June 14, 2023 20:14
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.

3 participants