Skip to content
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

Contributing clarification #917

Merged
merged 4 commits into from
May 16, 2014

Conversation

ekryski
Copy link
Contributor

@ekryski ekryski commented Apr 29, 2014

Since is the first time I've been contributing it took me a bit to figure out some of the steps. I made an attempt to clarify the contributing guide and be a bit more explicit in some areas.

@justinbmeyer
Copy link
Contributor

Should you be adding yourself to the list of heroes?

@ekryski
Copy link
Contributor Author

ekryski commented Apr 29, 2014

@justinbmeyer Could probably do that I guess.

@ccummings ccummings added this to the 2.1.1 milestone May 7, 2014
ccummings pushed a commit that referenced this pull request May 16, 2014
@ccummings ccummings merged commit 4ce353b into canjs:master May 16, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants