Skip to content

Conversation

@guybedford
Copy link

@guybedford guybedford commented Nov 21, 2024

This is the latest PR diff for the Stage 3 Source Phase Imports proposal, based to the import attributes updates on main.

This PR replaces #3094, since work moved back into the specification repo since then with further changes which are brought back here.

All review feedback on the previous PR was formerly also upstreamed, so that all review comments should be addressed. This sync was previously done in tc39/proposal-source-phase-imports#60 in April 2024, so included the latest comments.

To summarize the outstanding discussions from that PR here:

Formerly we had a review approval from @syg as well (#3094 (review)) down to some questions in the threads.

This PR forms the new base for all subsequent module harmony proposals going forward including Import Defer and the ESM Phase Imports proposals, as such it will be important to ensure it is maintained towards Stage 4 progression.

@guybedford guybedford force-pushed the source-phase-imports branch 2 times, most recently from 952c7d6 to 7468100 Compare November 21, 2024 01:08
@ljharb ljharb added normative change Affects behavior required to correctly evaluate some ECMAScript source text needs test262 tests The proposal should specify how to test an implementation. Ideally via github.com/tc39/test262 pending stage 4 This proposal has not yet achieved stage 4, but may otherwise be ready to merge. proposal This is related to a specific proposal, and will be closed/merged when the proposal reaches stage 4. labels Nov 21, 2024
@ljharb ljharb requested a review from a team November 21, 2024 01:24
@guybedford
Copy link
Author

Since previously, test-262 tests were added in tc39/test262#3980, tc39/test262#4252 and tc39/test262#4277.

@ljharb ljharb added has test262 tests and removed needs test262 tests The proposal should specify how to test an implementation. Ideally via github.com/tc39/test262 labels Nov 21, 2024
@guybedford guybedford changed the base branch from main to import-attributes November 21, 2024 16:20
@guybedford guybedford changed the base branch from import-attributes to main November 21, 2024 16:20
@guybedford guybedford force-pushed the source-phase-imports branch from 203c773 to 629223c Compare October 6, 2025 22:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

has test262 tests normative change Affects behavior required to correctly evaluate some ECMAScript source text pending stage 4 This proposal has not yet achieved stage 4, but may otherwise be ready to merge. proposal This is related to a specific proposal, and will be closed/merged when the proposal reaches stage 4.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants