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

Projects
None yet
3 participants
@ekryski
Contributor

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

This comment has been minimized.

Show comment
Hide comment
@justinbmeyer

justinbmeyer Apr 29, 2014

Contributor

Should you be adding yourself to the list of heroes?

Contributor

justinbmeyer commented Apr 29, 2014

Should you be adding yourself to the list of heroes?

@ekryski

This comment has been minimized.

Show comment
Hide comment
@ekryski

ekryski Apr 29, 2014

Contributor

@justinbmeyer Could probably do that I guess.

Contributor

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 added a commit that referenced this pull request May 16, 2014

@ccummings ccummings merged commit 4ce353b into canjs:master May 16, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment