Skip to content

Conversation

queengooborg
Copy link
Member

@queengooborg queengooborg commented Sep 24, 2025

This PR updates the CSS test generator to handle the new data structure of @webref/css v7.

A part of this upgrade involves flattening CSS types to get the enumerated values. In v6, Webref provided a values property which sometimes had these values already flattened out, but not always. v7 removed the values property, which now means that types had to be flattened. This creates a ton more tests as a result.

@queengooborg queengooborg temporarily deployed to mdn-bcd-collector-pr-2751 September 24, 2025 05:18 Inactive
@queengooborg queengooborg temporarily deployed to mdn-bcd-collector-pr-2751 September 24, 2025 13:10 Inactive
@queengooborg queengooborg temporarily deployed to mdn-bcd-collector-pr-2751 September 25, 2025 05:51 Inactive
…ing or encoding

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@queengooborg queengooborg temporarily deployed to mdn-bcd-collector-pr-2751 September 25, 2025 05:53 Inactive
@queengooborg queengooborg temporarily deployed to mdn-bcd-collector-pr-2751 September 25, 2025 05:55 Inactive
@queengooborg queengooborg marked this pull request as ready for review September 25, 2025 06:04
These are all defined in the spec for counter styles, but not in a way Webref can parse.
@queengooborg
Copy link
Member Author

Attached is the changelog from this update. @Elchi3, would you be down to take a look and tell me if you think this list of added (and just as importantly, removed) tests is alright, or do we want to try and fine-tune it more before including it in the next collector release?

css-changelog.md

@Elchi3
Copy link
Member

Elchi3 commented Sep 29, 2025

Thanks @queengooborg, can you present this at the next BCD call?

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.

2 participants