Skip to content

Remove test for markers changing with required-environments#299

Merged
konstin merged 1 commit intomainfrom
konsti/remove-markers-change-after-selection
Feb 18, 2026
Merged

Remove test for markers changing with required-environments#299
konstin merged 1 commit intomainfrom
konsti/remove-markers-change-after-selection

Conversation

@konstin
Copy link
Member

@konstin konstin commented Dec 8, 2025

This is failing CI in uv and I haven't figured out a fix yet: astral-sh/uv#16876 (comment)

This is failing CI in uv and I haven't figured out a fix yet: astral-sh/uv#16876 (comment)
@zanieb
Copy link
Member

zanieb commented Dec 8, 2025

Should we instead just filter this scenario out in uv?

@konstin
Copy link
Member Author

konstin commented Dec 11, 2025

Do we have a good way to do this sort of filtering?

@konstin
Copy link
Member Author

konstin commented Feb 18, 2026

Removing the test for now, they are blocking updating packse in uv (https://github.com/astral-sh/uv/actions/runs/22150080732/job/64038810687?pr=17881)

@konstin konstin merged commit 89c48cd into main Feb 18, 2026
8 checks passed
@konstin konstin deleted the konsti/remove-markers-change-after-selection branch February 18, 2026 17:43
@zanieb
Copy link
Member

zanieb commented Feb 18, 2026

I'm confused that you didn't just add filtering in uv, it seems like it would just be a continue in a loop somewhere? 🤷‍♀️

@konstin
Copy link
Member Author

konstin commented Feb 18, 2026

I'm not really following, do you mean like hardcoding the test name in the generation script? This test isn't particularly important.

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