Skip to content

Make commander a dev and build dependency and bump commander to latest version#1461

Open
AlynxZhou wants to merge 2 commits into
mozilla:masterfrom
AlynxZhou:bump-commander-version
Open

Make commander a dev and build dependency and bump commander to latest version#1461
AlynxZhou wants to merge 2 commits into
mozilla:masterfrom
AlynxZhou:bump-commander-version

Conversation

@AlynxZhou
Copy link
Copy Markdown
Contributor

Summary

Proposed change:

  • Make commander a dev and build dependency:

It is only used in precompile script, people who install nunjucks as a
dependency is unlikely to use precompile script.

  • Bump commander to latest version.

Checklist

I've completed the checklist below to ensure I didn't forget anything. This makes reviewing this PR as easy as possible for the maintainers. And it gets this change released as soon as possible.

It is only used in precompile script, people who install nunjucks as a
dependency is unlikely to use precompile script.
@AlynxZhou
Copy link
Copy Markdown
Contributor Author

Should I update the docs to tell people who needs precompile to run npm install --include dev?

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.

1 participant