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

Support for AMD and corrected browser #34

Closed
wants to merge 1 commit into from

Conversation

harryxue1999
Copy link

  • Added load support for AMD (such as requirejs)
  • Added load support for CMD (as well as NodeJS)
  • Corrected browser implementation (script not loaded by script tag may not have arc defined in window)

@jgravois
Copy link
Collaborator

jgravois commented May 9, 2022

hey @harryxue1999!

thank you for taking the time to propose this change and sorry for the belated response! turning arc.js into a UMD is still definitely an option but at this point it has also become pretty common for library authors to publish a bare ES module and let the consumer handle transpilation.

i'll need to chew on this a bit but i'm interested in more feedback if you (or anyone else) has opinions on the topic.

(maybe something like this? felixge/node-dateformat#183)

@jgravois
Copy link
Collaborator

jgravois commented Aug 15, 2024

a few more years have passed and i'm more convinced than ever that a native ESM rewrite would be the way to go. i don't have any plans to tackle anything like that myself, but a PR would definitely be welcome. 🙏

@jgravois jgravois closed this Aug 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants