Skip to content
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

Add feature for discouraged import assertions #2565

Merged
merged 2 commits into from
Jan 23, 2025

Conversation

ddbeck
Copy link
Collaborator

@ddbeck ddbeck commented Jan 22, 2025

Prompted by #2564.

This feature ought to be tombstoned (see #91 (comment) for discussion on that) but for now we have no-support compat data, so it's pretty straight forward to include it now, without doing anything special (excepting the spec URL).

@github-actions github-actions bot added feature definition Creating or defining new features or groups of features. tools and infrastructure Project internal tooling, such as linters, GitHub Actions, or repo settings labels Jan 22, 2025
features/import-assertions.yml Outdated Show resolved Hide resolved
],
[
"https://github.com/tc39/proposal-import-attributes/tree/abca60286360b47f9a6be25a28f489c2cb157beb",
"Allowed because import assertions were replaced in-place by import attributes. Remove this exception when javascript.statements.import.import_assertions is dropped from BCD in June 2026."
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice-to-have: a script that goes over this array and checks if the specs listed here are actually used in features. This way we don't need to remember to remove items.

@ddbeck ddbeck merged commit b0f1ef6 into web-platform-dx:main Jan 23, 2025
3 checks passed
@ddbeck ddbeck deleted the import-assertions branch January 23, 2025 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature definition Creating or defining new features or groups of features. tools and infrastructure Project internal tooling, such as linters, GitHub Actions, or repo settings
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants