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

Pull Request Template #1433

Closed
mweinelt opened this issue Jun 13, 2018 · 9 comments
Closed

Pull Request Template #1433

mweinelt opened this issue Jun 13, 2018 · 9 comments
Labels

Comments

@mweinelt
Copy link
Contributor

mweinelt commented Jun 13, 2018

We need a pull request template to improve on applying our guidelines and better able to track progress of pull requests.

@skorpy2009 and @rotanid wanted to take care of that. I recommend HackMD to collaborately build the text and neoraider recommended to use HTML comments to hide explanations etc..

Make use of our notes from the meeting here: https://md.darmstadt.ccc.de/gluon-device-support-policy#

@rotanid rotanid added the 3. topic: docs Topic: Documentation label Jun 13, 2018
@skorpy2009
Copy link
Member

This depends on #1438 & #1434

@mweinelt
Copy link
Contributor Author

GitHub allows for several different types of ISSUE_TEMPLATEs, probably the same for pull requests, so we don't need to cram everything into a single template.

https://blog.github.com/2018-05-02-issue-template-improvements/

via https://github.com/ffrgb/meshviewer/tree/develop/.github/ISSUE_TEMPLATE

@rotanid
Copy link
Member

rotanid commented Aug 8, 2018

@mweinelt you can only add a single PR template at the moment, as far as i can see it

EDIT: ok, they are possible, but not easily available to a user...

@mweinelt
Copy link
Contributor Author

mweinelt commented Aug 8, 2018

Sure, but better start somewhere, GitHub will follow at some point.

@rotanid
Copy link
Member

rotanid commented Aug 8, 2018

sure, feel free to do a pull request adding templates ;)

@mweinelt
Copy link
Contributor Author

mweinelt commented Aug 8, 2018

Great, let's do it collaboratively:

https://md.darmstadt.ccc.de/gluon-templates

@christf
Copy link
Member

christf commented Feb 27, 2019

which problem are we addressing with these templates?

@mweinelt
Copy link
Contributor Author

Lack of information and structure in pull requests. Like when you add a new device you'd get the device checklist by default.

The issue is that we currently can only have on PR template as it looks.

@mweinelt mweinelt unpinned this issue Mar 14, 2019
@mweinelt
Copy link
Contributor Author

mweinelt commented Mar 26, 2022

This did not pan out, which is fine, since nobody seems to be complaining about the current state of PR descriptions.

On top of that GitHub still does not support multiple PR templates and offering the device integration checklist as a default is a non-starter.

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

No branches or pull requests

4 participants