You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, loader.prototype.load will take a name and referrer and run the resolve hook just like loader.prototype.import.
Since the goal of loader.prototype.load is to provide a lower-level control over exactly what hook values are set for a specific module in the registry, it may be worth expecting the input to already be a resolved module key in the registry. This is how the previous loader specification was written as well - to not run normalize in loader.prototype.load.
The benefit of this is that we know for a fact then that there is no risk that resolve will alter the name given if we know the exact module name we want to set in the registry (say for a URL). If an unresolved name is needed, the loader.prototype.resolve function is available to use first manually if necessary.
The text was updated successfully, but these errors were encountered:
agreed, there are some things that will have to change in this regards. I faced a similar issue yesterday when working on the referral implementation. referral should be a key from the corresponding registry it seems.
Currently,
loader.prototype.load
will take aname
andreferrer
and run the resolve hook just likeloader.prototype.import
.Since the goal of
loader.prototype.load
is to provide a lower-level control over exactly what hook values are set for a specific module in the registry, it may be worth expecting the input to already be a resolved module key in the registry. This is how the previous loader specification was written as well - to not run normalize inloader.prototype.load
.The benefit of this is that we know for a fact then that there is no risk that
resolve
will alter the name given if we know the exact module name we want to set in the registry (say for a URL). If an unresolved name is needed, theloader.prototype.resolve
function is available to use first manually if necessary.The text was updated successfully, but these errors were encountered: