Skip to content

Conversation

@siketyan
Copy link
Member

@siketyan siketyan commented Jul 8, 2025

Summary

FileFeaturesResult.featuresSupported is defined as { [K in FeatureKind]?: SupportKind } but actually it's a Map object. Reverted some changes and added ts-expect-error for now. I'll fix the type definition later to match the actual value.

@siketyan siketyan requested review from a team and arendjr July 8, 2025 13:15
@siketyan siketyan self-assigned this Jul 8, 2025
@netlify
Copy link

netlify bot commented Jul 8, 2025

Deploy Preview for biomejs ready!

Name Link
🔨 Latest commit 80f8ebd
🔍 Latest deploy log https://app.netlify.com/projects/biomejs/deploys/686d1a0aa5e7e10008783ad1
😎 Deploy Preview https://deploy-preview-2664--biomejs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@siketyan siketyan marked this pull request as ready for review July 8, 2025 13:16
@siketyan siketyan changed the title fix: unexpected "Not supported" output fix(playground): unexpected "Not supported" output Jul 8, 2025
@dyc3
Copy link
Contributor

dyc3 commented Jul 8, 2025

Not the first time this has happened, I've run into this footgun before.

Copy link
Contributor

@arendjr arendjr left a comment

Choose a reason for hiding this comment

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

Sorry, my bad!

@siketyan
Copy link
Member Author

siketyan commented Jul 8, 2025

The type mismatch will be fixed in biomejs/biome#6781

@siketyan
Copy link
Member Author

siketyan commented Jul 8, 2025

I have already merged the fix for the type mismatch, closing this workaround.

@siketyan siketyan closed this Jul 8, 2025
@siketyan siketyan deleted the fix/playground-not-supported branch July 8, 2025 13:56
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