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 a pull request template which contains helpful information #633

Merged
merged 9 commits into from
Aug 22, 2023

Conversation

k00ni
Copy link
Collaborator

@k00ni k00ni commented Aug 16, 2023

I added a pull request template which contains important information about how we do things here. I'm tired of repeating myself all the time 😄, for instance asking people to add at least a unit test. After the merge, when opening the form to create a pull request, the new text will tell people all important things to know before creating a pull request.

My English is not the best, so please don't hesitate to suggest changes!

CC: @GreyWyvern @jzohrab

.github/pull_request_template.md Outdated Show resolved Hide resolved
.github/pull_request_template.md Outdated Show resolved Hide resolved
.github/pull_request_template.md Show resolved Hide resolved
k00ni and others added 5 commits August 21, 2023 16:31
Co-authored-by: Jérémy Benoist <j0k3r@users.noreply.github.com>
Co-authored-by: Jérémy Benoist <j0k3r@users.noreply.github.com>
Co-authored-by: Jérémy Benoist <j0k3r@users.noreply.github.com>
@k00ni
Copy link
Collaborator Author

k00ni commented Aug 22, 2023

@j0k3r can I merge?

@k00ni k00ni merged commit 81482e8 into master Aug 22, 2023
29 checks passed
@k00ni k00ni deleted the feature/add-pr-template-with-basic-intro branch August 22, 2023 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants