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 project types "yarn" and "pnpm" separate from "npm" #1862

Merged
merged 4 commits into from
Sep 19, 2023

Conversation

knu
Copy link
Contributor

@knu knu commented Sep 16, 2023

Yarn and pnpm are two popular alternatives to npm and you can tell which is used in a project by the existence of their respective lock files.

Plus, the build script is worth being a default "compile" command for npm-based projects.

  • The commits are consistent with our contribution guidelines
  • You've added tests (if possible) to cover your change(s)
  • All tests are passing (eldev test)
  • The new code is not generating bytecode or M-x checkdoc warnings
  • You've updated the changelog (if adding/changing user-visible functionality)
  • You've updated the readme (if adding/changing user-visible functionality)

@bbatsov bbatsov merged commit eaf2644 into bbatsov:master Sep 19, 2023
5 of 6 checks passed
@bbatsov
Copy link
Owner

bbatsov commented Sep 19, 2023

Looks good. Thanks!

@knu knu deleted the npm_project_types branch September 19, 2023 06:33
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.

None yet

2 participants