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
Import cycles have some interaction with star exports. Where one star export finds a cycle and another finds a binding, the binding should override and resolve successfully.
It was pointed out to me that typically a cycle will cause a crash elsewhere, when InitializeEnvironment detects it. But that shouldn't happen in the case where a star export is in the middle of the cycle. There is inconsistency in implementations on this point. I wonder if it should be in the test suite.
There are eshost outputs below, plus browser demos. The browser demos log/error to the console.
Import cycles have some interaction with star exports. Where one star export finds a cycle and another finds a binding, the binding should override and resolve successfully.
It was pointed out to me that typically a cycle will cause a crash elsewhere, when
InitializeEnvironment
detects it. But that shouldn't happen in the case where a star export is in the middle of the cycle. There is inconsistency in implementations on this point. I wonder if it should be in the test suite.There are eshost outputs below, plus browser demos. The browser demos log/error to the console.
Cycle on both sides, wrapped in named exports.
https://bojavou.github.io/star-export-cycle/biname/
https://github.com/bojavou/star-export-cycle/tree/main/biname
Cycle on one side, wrapped in named exports.
https://bojavou.github.io/star-export-cycle/uniname/
https://github.com/bojavou/star-export-cycle/tree/main/uniname
Cycle on both sides, wrapped in star exports.
https://bojavou.github.io/star-export-cycle/bistar/
https://github.com/bojavou/star-export-cycle/tree/main/bistar
Cycle on one side, wrapped in star exports.
This behaves identically to bistar.
https://bojavou.github.io/star-export-cycle/unistar/
https://github.com/bojavou/star-export-cycle/tree/main/unistar
The text was updated successfully, but these errors were encountered: