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

add CONTRIBUTING.md #31

Open
lukasheinrich opened this issue May 18, 2020 · 7 comments
Open

add CONTRIBUTING.md #31

lukasheinrich opened this issue May 18, 2020 · 7 comments
Assignees
Labels
docs Documentation related

Comments

@lukasheinrich
Copy link
Collaborator

No description provided.

@eduardo-rodrigues
Copy link
Member

@henryiii committed just yesterday a CONTRIBUTING file to https://github.com/scikit-hep/particle, with a format that is meant to be used by whoever wants to in Scikit-HEP, to have some uniformity. You could just use that and have this done in no time :-).

@henryiii
Copy link
Member

We might want to expand it a bit, but the important part is the link to the central location on the scikit-hep.org page, which we can update.

@matthewfeickert
Copy link
Member

but the important part is the link to the central location on the scikit-hep.org page, which we can update.

Unless every Scikit-HEP library is going to have the same setup.py extras, shouldn't the CONTRIBUTING be tailored to each project? It make sense to have a template that we can all use, but having a single file for all projects seems like it would be problematic.

Maybe I misunderstood though. Are you just saying that we should all add the

See the Scikit-HEP Developer introduction for a detailed description of best practices for developing Scikit-HEP packages.

to the top of our CONTRIBUTING files?

@henryiii
Copy link
Member

I really think we should try to standardize on a minimal set of extras we include, like "all", "dev", and "test". But the single page is for more detailed but general setup, and the CONTRIBUTING template and any customizations (or a completely new file) is for the specifics of each project (like what extras are present, etc).

So yes, each CONTRIBUTING is expected to be different, and the template is a suggestion. However, they all should probably link to the general guidelines at the top, like that one.

@eduardo-rodrigues
Copy link
Member

Hello @amangoel185. If I'm not mistaken you have been updating several org packages with contributing files. Can you point me to a very recent example so that we add one CONTRIBUTING in here? Advance thanks.

@amangoel185
Copy link

Hi @eduardo-rodrigues, I haven't made one from scratch recently, but I like the ones present in uproot-browser and Hist.

The basic idea is to have something similar to what @henryiii and @matthewfeickert mentioned.

On the top:

See the Scikit-HEP Developer introduction for a detailed description of best practices for developing Scikit-HEP packages.

And then sections for Quick development, PyPI, Conda, Post setup, Testing, Pre-commit etc.

I also think that there were some advantages to having the CONTRIBUTING.md in the .github directory rather than root, @henryiii might know more!

@eduardo-rodrigues
Copy link
Member

Thanks @amangoel185. I will certainly have a look soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation related
Projects
None yet
Development

No branches or pull requests

5 participants