Skip to content

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

Merged
merged 9 commits into from
Apr 4, 2025

Conversation

tunetheweb
Copy link
Contributor

@tunetheweb tunetheweb commented Apr 2, 2025

Summary

Some fixes from #26257 :

Test results and supporting details

Related issues

#26257
#24518

@github-actions github-actions bot added data:http Compat data for HTTP features. https://developer.mozilla.org/docs/Web/HTTP data:html Compat data for HTML elements. https://developer.mozilla.org/docs/Web/HTML size:m [PR only] 25-100 LoC changed labels Apr 2, 2025
@github-actions github-actions bot added size:l [PR only] 101-1000 LoC changed and removed size:m [PR only] 25-100 LoC changed labels Apr 4, 2025
Copy link
Member

@Elchi3 Elchi3 left a 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

@tunetheweb
Copy link
Contributor Author

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.

@github-actions github-actions bot added size:m [PR only] 25-100 LoC changed and removed size:l [PR only] 101-1000 LoC changed labels Apr 4, 2025
@tunetheweb tunetheweb requested a review from Elchi3 April 4, 2025 11:28
Copy link
Member

@Elchi3 Elchi3 left a 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 👍

@Elchi3 Elchi3 merged commit 90a4a8b into mdn:main Apr 4, 2025
5 checks passed
@mdn-bot mdn-bot mentioned this pull request Apr 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:html Compat data for HTML elements. https://developer.mozilla.org/docs/Web/HTML data:http Compat data for HTTP features. https://developer.mozilla.org/docs/Web/HTTP size:m [PR only] 25-100 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants