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

✨ Export gitmojis as CJS and ESM #1280

Merged
merged 8 commits into from
Jan 18, 2023
Merged

Conversation

carloscuesta
Copy link
Owner

@carloscuesta carloscuesta commented Jan 16, 2023

Description

Hey! πŸ‘‹πŸΌ

This PR exports the gitmojis package both as a CJS and ESM modules.

Drawbacks

Fixed in #1282 (refresh is a bit slower but works!)

Linked issues

Fixes #1279

Eventually we will deprecate CJS but for now keeping both versions for backwards compatibility.

@vercel
Copy link

vercel bot commented Jan 16, 2023

Deployment failed with the following error:

Resource is limited - try again in 4 hours (more than 100, code: "api-deployments-free-per-day").

@carloscuesta carloscuesta force-pushed the export-gitmojis-cjs-and-esm branch from 30265a0 to a4b7164 Compare January 17, 2023 22:00
@vercel
Copy link

vercel bot commented Jan 17, 2023

The latest updates on your projects. Learn more about Vercel for Git β†—οΈŽ

Name Status Preview Comments Updated
gitmoji βœ… Ready (Inspect) Visit Preview πŸ’¬ Add your feedback Jan 18, 2023 at 6:30PM (UTC)

@carloscuesta carloscuesta force-pushed the export-gitmojis-cjs-and-esm branch from 659a3a8 to 6b4c969 Compare January 17, 2023 22:28
@carloscuesta carloscuesta marked this pull request as ready for review January 17, 2023 22:30
@carloscuesta carloscuesta requested a review from vhoyer January 17, 2023 22:31
@carloscuesta carloscuesta changed the title ✨ Export gitmojis as cjs and esm ✨ Export gitmojis as CJS and ESM Jan 17, 2023
@carloscuesta carloscuesta changed the title ✨ Export gitmojis as CJS and ESM ✨ Export gitmojis as CJS and ESM Jan 17, 2023
@vhoyer
Copy link
Collaborator

vhoyer commented Jan 18, 2023

nice, but I think #1282 would come in handy, especially to the help of people new to the project

@carloscuesta
Copy link
Owner Author

Of course @vhoyer!! Thanks for raising that PR, I added nodemon to the gitmojis package but I was not aware of the parallel πŸ˜› option, so it was not working as expected! πŸ™πŸΌ

@carloscuesta
Copy link
Owner Author

Let's ship this πŸš€

@carloscuesta carloscuesta merged commit 8fd367f into master Jan 18, 2023
@carloscuesta carloscuesta deleted the export-gitmojis-cjs-and-esm branch January 18, 2023 18:33
@vhoyer
Copy link
Collaborator

vhoyer commented Jan 18, 2023

Glad to help

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support CJS and ESM exports on the gitmojis package
2 participants