Skip to content

fix: ESM integration notes for latest binary changes#204

Closed
guybedford wants to merge 1 commit intomainfrom
esm-upgrade
Closed

fix: ESM integration notes for latest binary changes#204
guybedford wants to merge 1 commit intomainfrom
esm-upgrade

Conversation

@guybedford
Copy link
Collaborator

Adds the omitted ESM integration update for #198.

@alexcrichton
Copy link
Collaborator

Oh apologies for jumping the gun! I wasn't thinking that the Explainer.md was entirely sorted out though as I hadn't touched it at all just yet. @lukewagner should be working on more expansive changes to Explainer.md, but otherwise this looks good to me too 👍

@guybedford
Copy link
Collaborator Author

guybedford commented Jun 15, 2023

Thanks for clarifying, was just a bit surprised to see it not being addressed after my comment, completely understood that PR had been stood open a while and the need to move forward now as well.

@lukewagner
Copy link
Member

#205 I believe subsumes this PR, marking the particular paragraph as a "TODO".

@guybedford
Copy link
Collaborator Author

Thanks, that covers it fine as well for now.

@guybedford guybedford closed this Jun 20, 2023
@guybedford guybedford deleted the esm-upgrade branch June 20, 2023 21:15
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.

3 participants