Added Contributing file with information - #226
Merged
Merged
Conversation
una
reviewed
Oct 20, 2016
| ## Procedure 1 | ||
| 1. Create an [issue](https://github.com/una/CSSgram/issues) |
Owner
There was a problem hiding this comment.
I'd say look for an issue first becuase a lot of them are already made
| 2. Clone the fork onto your system | ||
| 3. `npm install` dependencies (must have Node installed) | ||
| 4. Run `gulp` to compile CSS and the test site | ||
| 5. Make changes and check the test site with your changes (see file structure outline below) |
Owner
There was a problem hiding this comment.
Missing file structure outline here (also mention to edit the .scss file related to the filter)
una
commented
Oct 20, 2016
Owner
Hey -- have you taken a look at any of these comments yet? |
Bobeta
commented
Oct 23, 2016
ContributorAuthor
I made the requested changes 👍 |
una
commented
Oct 27, 2016
Owner
There's also slightly different information here: #242 with this being merged, please include that as well |
Bobeta
commented
Oct 31, 2016
ContributorAuthor
There you go 👍 an idea would be to just merge it and update it on the fly as new things come along, instead of me maintaining the docs haha 😆 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I happened to miss the contribution section in the Readme while looking through the project. As many others do, I was looking for the CONTRIBUTING.md file in the repository, but couldn't find it. I suggest we add it with this pull request 👍