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

Adding CONTRIBUTING file to BF repo #1072

Merged
merged 2 commits into from May 1, 2014

Conversation

hflynn
Copy link
Member

@hflynn hflynn commented Apr 30, 2014

Ticket 10030 fix - adding a contributing file which is pointed out to GH users who open an issue or PR against the BF repo.

@hflynn
Copy link
Member Author

hflynn commented Apr 30, 2014

@hflynn
Copy link
Member Author

hflynn commented Apr 30, 2014

Travis build is failing on the CPP build so not related to this PR.

@joshmoore
Copy link
Member

Re-launched travis. Perhaps time for a check, @sbesson, to detect when a file is docs only and not run the other builds? 😄

@melissalinkert
Copy link
Member

Only minor thing is that I'm not sure we want to advertise a 2 business day review policy, but I'm happy to be outvoted. All looks fine to me otherwise.

@hflynn
Copy link
Member Author

hflynn commented May 1, 2014

I did wonder about that given we have less reviewers on the BF side, happy to change it to indicate we get to things as quickly as we can instead?

@qidane
Copy link
Contributor

qidane commented May 1, 2014

Reads fine to me. All links worked. Good to merge.

@hflynn
Copy link
Member Author

hflynn commented May 1, 2014

--rebased-to #1076

melissalinkert added a commit that referenced this pull request May 1, 2014
Adding CONTRIBUTING file to BF repo
@melissalinkert melissalinkert merged commit 001c8a5 into ome:dev_5_0 May 1, 2014
@hflynn hflynn deleted the contributing-file branch October 7, 2014 10:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants