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.

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

Add esm build. Fixes #2371 #2372

Merged
merged 3 commits into from
Oct 7, 2021
Merged

Conversation

sidharthv96
Copy link
Member

@sidharthv96 sidharthv96 commented Sep 30, 2021

📑 Summary

Add an esm version to build output

Resolves #2371

📏 Design Decisions

Without an esm version, live editor was failing.

📋 Tasks

Make sure you

  • 📖 have read the contribution guidelines
  • 💻 have added unit/e2e tests (if appropriate)
  • 🔖 targeted develop branch

@github-actions github-actions bot added the Discussion Discussion which may lead to separate issues or PRs label Sep 30, 2021
@sidharthv96 sidharthv96 self-assigned this Sep 30, 2021
@mmorel-35
Copy link
Contributor

Have you seen https://github.com/msgpack/msgpack-javascript ?
The way it handles multiple output formats particularly.
Can it be a good inspiration to reduce the number of webpack config files?

@sidharthv96
Copy link
Member Author

Thanks @mmorel-35, I did a quick search for multiple outputs from a single webpack config when I saw the array export, but didn't get any good examples.

Converted.

@sidharthv96
Copy link
Member Author

sidharthv96 commented Oct 1, 2021

Please don't merge this yet, facing some issues when building in CI.
Fixed.

Live editor preview deployed: https://deploy-preview-411--mermaidjs.netlify.app

Copy link
Collaborator

@ashishjain0512 ashishjain0512 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for taking a look at this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Discussion Discussion which may lead to separate issues or PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Live editor build broken
3 participants