html icon indicating copy to clipboard operation
html copied to clipboard

Confusing xref for HostResolveImportedModule

Open Ms2ger opened this issue 2 years ago • 3 comments

The definition of HostResolveImportedModule links to ES but implements the version in import assertions, which took me a minute to find. It would be better to link to the latter, I think.

Ms2ger avatar Sep 05 '22 12:09 Ms2ger

Although this is a minor editorial change and would probably be fine, it's worth noting there is a proposal to be presented at the next TC39 meeting for refactoring all the host module hooks into a single hook, which which ultimately invalidate such a change anyway.

Jamesernator avatar Sep 05 '22 14:09 Jamesernator

A PR from anyone fixing this would be very welcome in the meantime.

domenic avatar Sep 07 '22 09:09 domenic

I would be interested in picking up this small, soon obsolete contribution, in order to practice my PR on some open source code!

katmlane avatar Sep 18 '22 03:09 katmlane