Skip to content

Conversation

@mrlubos
Copy link
Contributor

@mrlubos mrlubos commented Feb 20, 2018

Previous versions used --dev flag to install only devDependencies. That has now been deprecated in favour of --only={prod[uction]|dev[elopment]}.

@mrlubos
Copy link
Contributor Author

mrlubos commented Feb 20, 2018

Hello! 👋

My first PR, I found one more typo so far reading through the documentation, let me know if you don't want to have pull requests for these things. Thanks!

Previous versions used --dev flag to install only devDependencies. That has now been deprecated in favour of --only={prod[uction]|dev[elopment]} (see https://docs.npmjs.com/cli/install).
Copy link
Member

@Elchi3 Elchi3 left a comment

Choose a reason for hiding this comment

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

Hello, this looks good to me.
Thanks for contributing to mdn-browser-compat-data!

@Elchi3 Elchi3 added the docs Issues or pull requests regarding the documentation of this project. label Feb 21, 2018
@Elchi3 Elchi3 merged commit 5a6682d into mdn:master Feb 21, 2018
@mrlubos
Copy link
Contributor Author

mrlubos commented Feb 21, 2018

Hi @Elchi3, thank you! I have picked a few features to convert in the meantime, I will tag you once I submit the first PR to ensure I followed the correct format so I can contribute more.

dontcallmedom pushed a commit to dontcallmedom/browser-compat-data that referenced this pull request Mar 17, 2018
Previous versions used --dev flag to install only devDependencies. That has now been deprecated in favour of --only={prod[uction]|dev[elopment]} (see https://docs.npmjs.com/cli/install).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Issues or pull requests regarding the documentation of this project.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants