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 a CONTRIBUTING.md to repository #171

Merged
merged 1 commit into from Mar 23, 2015
Merged

Adding a CONTRIBUTING.md to repository #171

merged 1 commit into from Mar 23, 2015

Conversation

jeremyf
Copy link
Contributor

@jeremyf jeremyf commented Mar 4, 2015

This is a pointer to a stubbed out contributing document. I don't
know if it is the correct point of reference, but it is something
I would like to see as a developer.

[skip ci]

@runlevel5
Copy link
Member

@jeremyf Normally CONTRIBUTING outline the tech-side contribution guideline, for the sake of the convenience, I'd prefer we have the content ready within the file rather than redirecting users to another page

@jeremyf
Copy link
Contributor Author

jeremyf commented Mar 4, 2015

@joneslee85 Agreed. I have a script for pushing out a contributing document to each repository. I believe it is project centric but would be easy to adapt; Should likely live in lotus-docs.

@jeremyf
Copy link
Contributor Author

jeremyf commented Mar 4, 2015

I'm more interested in opening up the conversation. I added a conversation issue on lotus-docs lotus/docs#18.

@jodosha
Copy link
Member

jodosha commented Mar 4, 2015

I agree with @joneslee85 😄

@runlevel5
Copy link
Member

@jeremyf to me we should maintain 2 versions, the long version which resides in docs repo would go into details the workflows whilst the CONTRIBUTING.md here should be related purely to style guideline , I kinda like the one of Spree https://github.com/spree/spree/blob/master/CONTRIBUTING.md

@runlevel5
Copy link
Member

@jeremyf can you please kickstart this whenever you can? much thanks

@jeremyf
Copy link
Contributor Author

jeremyf commented Mar 13, 2015

I'm looking at Spree's CONTRIBUTING document and I like it as a baseline. One thing I'm concerned about is copying the CONTRIBUTING document verbatim; One I'd need to include a license, and two not all of it is applicable.

@runlevel5
Copy link
Member

@jeremyf this looks a good starting point, let me know if you want me to merge it now

@jeremyf
Copy link
Contributor Author

jeremyf commented Mar 13, 2015

A few +1s would be great

On Thursday, March 12, 2015, Trung Lê notifications@github.com wrote:

@jeremyf https://github.com/jeremyf this looks a good starting point,
let me know if you want me to merge it now


Reply to this email directly or view it on GitHub
#171 (comment).

Jeremy Friesen
takeonrules.com

@thecatwasnot
Copy link

👍

@AlfonsoUceda
Copy link
Contributor

👍 I think this file should be extrapolated to other lotus gems

@jeremyf
Copy link
Contributor Author

jeremyf commented Mar 16, 2015

@AlfonsoUceda Once "ratified" and modified, I'd love to push that out. I have a script for pushing updates to CONTRIBUTING to all repositories of an organization https://github.com/projecthydra/hydra/blob/master/script/push_contributing_document.rb; It could be used and modified to automate that process (if there is a perceived volatility in the CONTRIBUTING document)

@jeremyf
Copy link
Contributor Author

jeremyf commented Mar 16, 2015

I'd love @jodosha to weigh in as well.

@AlfonsoUceda
Copy link
Contributor

@jeremyf ok ;)

@jeremyf
Copy link
Contributor Author

jeremyf commented Mar 17, 2015

Or you know…go ahead and merge it!

@runlevel5
Copy link
Member

@jeremyf till this day, I still have no clue about what Lotus idioms are and the proper way to document the method. Maybe it's something that you could help with? I would love to adopt a strict code style if possible.

@jeremyf
Copy link
Contributor Author

jeremyf commented Mar 17, 2015

@joneslee85 If we are after a strict code style, then I would advocate that our builds should include Rubocop enforcement.

I've been doing it for our application and am please with the results. Applying it to a project that is already started can be a bit of busy work; But there are ways to inch toward that.

This is our configuration https://github.com/ndlib/sipity/blob/master/.hound.yml (we were using Hound at one point, but abandoned it as Rubocop enforces things as part of rake).

@runlevel5
Copy link
Member

@jeremyf I don't mind using Rubocop. What do you think @jodosha?

@jeremyf
Copy link
Contributor Author

jeremyf commented Mar 23, 2015

@joneslee85 based on a Gitter convesation, Rubocop is going to be angry; What I would like to see happen is that the contributing document be merged with "issues" being created to address any concerns.

A wise software developer once told me all we are doing is improving our approximations; So lets improve the documents by merging this pull request and (not &&) making an issue or two to address immediate shortcomings.

runlevel5 pushed a commit that referenced this pull request Mar 23, 2015
Adding a CONTRIBUTING.md to repository
@runlevel5 runlevel5 merged commit b04a50f into hanami:master Mar 23, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants