Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

"Module level directives cause errors when bundled, 'use jessie' was ignored." #2704

Closed
dckc opened this issue Mar 23, 2021 · 3 comments
Closed

"Module level directives cause errors when bundled, 'use jessie' was ignored." #2704

dckc opened this issue Mar 23, 2021 · 3 comments
Assignees
@michaelfig
Labels
bug Something isn't working

Comments

@dckc
Copy link
Member

dckc commented Mar 23, 2021

Describe the bug

'use jessie' seems to lead to warnings; perhaps in a bundling step?

To Reproduce

Steps to reproduce the behavior:
in packages/zoe, run yarn test:xs-unit. There are probably simpler ways, but that's the way it came up just now.

Additional context

This warning is most likely a consequence of #2606.

cc @michaelfig @erights @katelynsills
@dckc dckc added the bug Something isn't working label Mar 23, 2021
@dckc
Copy link
Member Author

dckc commented Mar 26, 2021

@michaelfig notes plans to migrate from use jessie to an import declaration.

dckc added a commit that referenced this issue Apr 21, 2021
@dckc
fix(ERTP): avoid jessie warning
127c8a6 
Let's hold off on `use jessie` until #2704 is addressed.
@dckc dckc mentioned this issue Apr 21, 2021
Merged
dckc added a commit that referenced this issue Apr 21, 2021
@dckc
fix(ERTP): avoid jessie warning
fa68a8a 
Let's hold off on `use jessie` until #2704 is addressed.
@dckc
Copy link
Member Author

dckc commented Apr 22, 2021

fixed in #2938

michaelfig pushed a commit that referenced this issue Apr 22, 2021
@dckc @michaelfig
fix(ERTP): avoid jessie warning
f8b86b4 
Let's hold off on `use jessie` until #2704 is addressed.
michaelfig pushed a commit that referenced this issue Apr 22, 2021
@dckc @michaelfig
fix(ERTP): avoid jessie warning
fe9319b 
Let's hold off on `use jessie` until #2704 is addressed.
@erights
Copy link
Member

erights commented Apr 22, 2021

In light of #2938 , should this be closed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@michaelfig michaelfig

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@dckc @erights @michaelfig