Skip to content

registry: Revert "fix(registry): fix biome test to handle version prefix"#7586

Merged
jdx merged 2 commits intojdx:mainfrom
risu729:revert-7585-fix/biome-test
Jan 7, 2026
Merged

registry: Revert "fix(registry): fix biome test to handle version prefix"#7586
jdx merged 2 commits intojdx:mainfrom
risu729:revert-7585-fix/biome-test

Conversation

@risu729
Copy link
Contributor

@risu729 risu729 commented Jan 7, 2026

Reverts #7585

Thank you for fixing the aqua-registry. I think that will fix the issue, and this test change isn't required.
Please close this revert PR if you prefer to keep it as a fallback.


Note

Reverts the recent change to the Biome tool test.

  • In registry.toml, updates tools.biome test to expect "Version: {{version}}" (removes split(pat="@") | last).

Written by Cursor Bugbot for commit ab9aae2. This will update automatically on new commits. Configure here.

Copilot AI review requested due to automatic review settings January 7, 2026 06:55
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR reverts a previous fix that added version prefix handling for the biome tool test. The revert is warranted because the underlying issue was fixed in the aqua-registry itself, making the workaround in the test unnecessary.

Key Changes:

  • Reverts the biome test from using split(pat="@") to handle version prefixes back to the standard {{version}} template

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@risu729 risu729 changed the title Revert "fix(registry): fix biome test to handle version prefix" registry: Revert "fix(registry): fix biome test to handle version prefix" Jan 7, 2026
@jdx jdx enabled auto-merge (squash) January 7, 2026 14:28
@jdx jdx merged commit ce6b4ff into jdx:main Jan 7, 2026
38 of 41 checks passed
@risu729 risu729 deleted the revert-7585-fix/biome-test branch January 7, 2026 14:42
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.

3 participants