-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Fix link rel compression dictionary #26375
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix link rel compression dictionary #26375
Conversation
…theweb/browser-compat-data into fix-link-rel-compression-dictionary
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If a feature is created in web-features, we sync the tagging to BCD automatically. The build fails as the web-feature ID doesn't exist yet (but will once dependabot updates BCD's web-feature version and we sync the tags). For new features, I would always recommend to author things in web-features as that should be the source of truth. The docs for this live here: https://github.com/web-platform-dx/web-features/blob/main/docs/bcd-sync.md
Yeah it's already added in there. Just waiting for the next version which was released, but hasn't come through to this repo yet. I have removed the web-feature additions then if they will auto sync, but still have the other changes in this PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, I ran tag syncing in #26404.
The other changes LGTM 👍
Summary
Some fixes from #26257 :
compression-dictionary is a
link relattribute, not a
linkattribute. Also removed the
expects` one that caused this confustion (added in error in fetchPriority enabled by default in Firefox 132 #24518 I believe).Test results and supporting details
Related issues
#26257
#24518