Skip to content

Conversation

@stan-dot
Copy link
Contributor

as in the issue #26 (comment)

upgrade to 1.25 will make pants test :: fail, so that is as far as we can go now

Copy link
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.

Thanks for this! I think it would be good to have the devcontainer piece in a separate PR though.

@stan-dot
Copy link
Contributor Author

done @benjyw

@benjyw
Copy link
Contributor

benjyw commented Oct 30, 2025

I notice that you pushed the dev container change back onto this or?

@benjyw
Copy link
Contributor

benjyw commented Oct 31, 2025

@stan-dot I can merge this once you revert the devcontainer commit, which somehow got pushed again. Thanks!

This reverts commit 02ff0ad.
@stan-dot
Copy link
Contributor Author

fixed, that was my mistake

@benjyw benjyw merged commit 1e83009 into pantsbuild:main Nov 11, 2025
2 checks passed
@benjyw
Copy link
Contributor

benjyw commented Nov 11, 2025

Thanks @stan-dot ! If you want to add the devcontainer in a separate PR, feel free!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants