-
Notifications
You must be signed in to change notification settings - Fork 14
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
NPM and Grunt related changes #4
Conversation
I am not sure, but I don't think these changes have anything to do with the failing CI tests. Do let me know if I need to change stuff to make them pass. |
Got confused between repos. Need to fix the devDependencies patch. |
This makes setting up easier. You can just run 'npm install' and then 'grunt'. We need to ignore the 'node_modules' directory as its managed by npm using the package.json file, no need for this to be checked in.
Fixed. |
I have some major changes in design in mind for this plugin |
That's great! I've been thinking of using this at a bunch of places myself. These changes are only build related and should be helpful in any case. |
This should make setting up the project easier. Unfortunately https://github.com/wpreadme2markdown/wp-readme-to-markdown doesn't package for npm, and so we'll have to make that upstream change first.