Skip to content

Conversation

@snyk-will
Copy link
Contributor

@snyk-will snyk-will commented Jan 6, 2026

Pull Request Submission Checklist

  • Follows CONTRIBUTING guidelines
  • Commit messages
    are release-note ready, emphasizing
    what was changed, not how.
  • Includes detailed description of changes
  • Contains risk assessment (Low | Medium | High)
  • Highlights breaking API changes (if applicable)
  • Links to automated tests covering new functionality
  • Includes manual testing instructions (if necessary)
  • Updates relevant GitBook documentation (PR link: ___)
  • Includes product update to be announced in the next stable release notes

What does this PR do?

In the latest version of the os-flows extension we have introduced a registry feature flag and removed the need for an environment variable and the experimental flag. This PR adds an acceptance test to check the implementation of the feature flag and the correct routing of the uv workflow through os-flows to the unified test API.

What's the product update that needs to be communicated to CLI users?

None, the feature has not yet reached closed beta

Risk assessment (Low)?

Low, just acceptance tests

What are the relevant tickets?

https://snyksec.atlassian.net/browse/UNIFY-1117
https://snyksec.atlassian.net/browse/UNIFY-1118

@snyk-will snyk-will force-pushed the chore/bump-os-uv-acceptance branch 3 times, most recently from e8ad3cb to c87b073 Compare January 8, 2026 13:05
@snyk-will snyk-will marked this pull request as ready for review January 8, 2026 13:57
@snyk-will snyk-will requested review from a team as code owners January 8, 2026 13:57
@snyk-will snyk-will force-pushed the chore/bump-os-uv-acceptance branch from c87b073 to 13cb05f Compare January 8, 2026 13:59
@CatalinSnyk
Copy link
Collaborator

The PR title mentions a bump for os-flows, but I'm guessing that is done though #6419? 🤔

@snyk-will snyk-will changed the title chore: bump os-flows + uv acceptance test chore: uv acceptance test Jan 8, 2026
@snyk-will
Copy link
Contributor Author

@CatalinSnyk that PR is unrelated to my change actually. I think os flows beat me to the update so the version I needed is actually already merged, I can update pr title

@snyk-will snyk-will merged commit 07e643f into main Jan 8, 2026
7 checks passed
@snyk-will snyk-will deleted the chore/bump-os-uv-acceptance branch January 8, 2026 15:39
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.

4 participants