Skip to content

Conversation

@Alex7Li
Copy link
Contributor

@Alex7Li Alex7Li commented Jul 14, 2020

Our package json has a lot of missing stuff which would make the library look more professional, see #5. That commit can't ever get merged since the contributor did not sign the CLA, though. So that's one reason for this commit. The other reason is that, by using the scripts property of the package json, we can simplify the build instructions in the README by a lot. (Bonus: It doesn't affect the current workflow if you want to ignore it). Scripts run by npm or yarn automatically add your project's node_modules/.bin/ folder to the $PATH before it runs the command, so there is no need for a global install. I think that is pretty cool and good.
Also I updated the version from 0.1.0 to 1.0.0 since we have been adding features and making a major update.

Copy link
Contributor

@kjgalvan kjgalvan left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@durbanie durbanie left a comment

Choose a reason for hiding this comment

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

Thanks Alex!

@zikas zikas merged commit a558bc0 into google:master Jul 22, 2020
@Alex7Li Alex7Li deleted the updatePackageJSON branch July 22, 2020 19:05
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.

4 participants