Skip to content

Add defaultLibrary semantic token selector aliases#754

Merged
vinistock merged 2 commits intoShopify:mainfrom
a-lavis:add-defaultlibrary-semantic-selector
Mar 17, 2026
Merged

Add defaultLibrary semantic token selector aliases#754
vinistock merged 2 commits intoShopify:mainfrom
a-lavis:add-defaultlibrary-semantic-selector

Conversation

@a-lavis
Copy link
Contributor

@a-lavis a-lavis commented Mar 13, 2026

What

Updates the Spinel Light/Dark themes to support: Shopify/ruby-lsp#4005

In the future, we can consider removing the variable.default_library selector, but I think its worth keeping for now for backwards compatibility until everyone has updated to a new version of ruby-lsp with the above change.

Why

See the above PR's description for more details.

@a-lavis a-lavis marked this pull request as ready for review March 13, 2026 18:00
@a-lavis a-lavis requested a review from a team as a code owner March 13, 2026 18:00
@a-lavis
Copy link
Contributor Author

a-lavis commented Mar 13, 2026

Just signed the CLA!

Copy link
Member

@vinistock vinistock left a comment

Choose a reason for hiding this comment

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

Regarding keeping the old default_library, I don't think it was actually doing anything. It's not a support semantic token modifiers, so unless the extension explicitly contributes it, it will be ignored.

I think we can already remove it in this PR.

@vinistock vinistock added the bugfix This pull request fixes a bug label Mar 17, 2026
@a-lavis
Copy link
Contributor Author

a-lavis commented Mar 17, 2026

I think we can already remove it in this PR.

Done in 3cbeaf9 - thank you!

@vinistock vinistock merged commit a8fb621 into Shopify:main Mar 17, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix This pull request fixes a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants