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.

Sign up for GitHub

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

Jump to bottom

Document PR scope & size #10174

Merged
merged 1 commit into from
Apr 17, 2017
Merged

Document PR scope & size #10174

merged 1 commit into from
Apr 17, 2017

Conversation

colemanw
Copy link
Member

No description provided.

@colemanw
Copy link
Member Author

@eileenmcnaughton I think this is what you were going for when you asked for smaller PRs.

@colemanw
Copy link
Member Author

@civicrm-builder retest this please

@seancolsen
Copy link
Contributor

There was some discussion on MM about where to add this content. @colemanw I think this is a fine place for now. Looks good!

@eileenmcnaughton eileenmcnaughton merged commit 818b494 into master Apr 17, 2017
@eileenmcnaughton eileenmcnaughton deleted the colemanw-contributing branch April 17, 2017 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
master
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@colemanw @seancolsen @eileenmcnaughton @civicrm-builder