Skip to content
This repository was archived by the owner on Mar 23, 2024. It is now read-only.

Docs: Contributing - more project and process details#643

Merged
mrjoelkemp merged 1 commit intojscs-dev:masterfrom
mrjoelkemp:contributing
Sep 21, 2014
Merged

Docs: Contributing - more project and process details#643
mrjoelkemp merged 1 commit intojscs-dev:masterfrom
mrjoelkemp:contributing

Conversation

@mrjoelkemp
Copy link
Member

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling d967d72 on mrjoelkemp:contributing into 0b43820 on jscs-dev:master.

@mrjoelkemp mrjoelkemp force-pushed the contributing branch 3 times, most recently from 7caef22 to 332bde6 Compare September 20, 2014 22:09
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 7caef22 on mrjoelkemp:contributing into 0b43820 on jscs-dev:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 332bde6 on mrjoelkemp:contributing into 0b43820 on jscs-dev:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling a037ca3 on mrjoelkemp:contributing into 0b43820 on jscs-dev:master.

@mrjoelkemp mrjoelkemp force-pushed the contributing branch 2 times, most recently from 3d21493 to c2d9a02 Compare September 20, 2014 22:23
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 3d21493 on mrjoelkemp:contributing into 0b43820 on jscs-dev:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling c2d9a02 on mrjoelkemp:contributing into 0b43820 on jscs-dev:master.

@mrjoelkemp mrjoelkemp changed the title *NOT READY FOR MERGE* Docs: Contributing - more project and process details Docs: Contributing - more project and process details Sep 20, 2014
@mrjoelkemp
Copy link
Member Author

@markelog @mikesherov @pomeh Thoughts on the added details? More background information on the AST structure and key parts of the codebase (like files in lib/) are still missing, but I'd like to get this 1st draft landed for us to point to when reviewing PRs or responding to issues.

@pomeh
Copy link
Contributor

pomeh commented Sep 20, 2014

@mrjoelkemp this sounds great to me 👍

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 8263b45 on mrjoelkemp:contributing into 0b43820 on jscs-dev:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 8263b45 on mrjoelkemp:contributing into 0b43820 on jscs-dev:master.

@mikesherov
Copy link
Contributor

LGTMMike Sherov

On Sat, Sep 20, 2014 at 8:15 PM, Coveralls notifications@github.com
wrote:

Coverage Status

Coverage remained the same when pulling 8263b45 on mrjoelkemp:contributing into 0b43820 on jscs-dev:master.

Reply to this email directly or view it on GitHub:
#643 (comment)

@mrjoelkemp mrjoelkemp merged commit 8263b45 into jscs-dev:master Sep 21, 2014
@mrjoelkemp mrjoelkemp deleted the contributing branch September 21, 2014 00:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants