-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Github adjustments #43
Conversation
@@ -0,0 +1,9 @@ | |||
## Changes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this template have:
- What problem is this change solving? (if there is an issue, link to it and no need to describe)
- What's the solution being applied?
- How should I read the files in this change/which are the most important ones?
I definitely don't want it to be heavyweight but I find that I often have to reverse engineer all these things from the flat code/files which, if there is more than 1 reviewer, can be a time drain.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whoops merge too quickly. Will add that tmw
labels: enhancement | ||
|
||
--- | ||
## Describe the solution you'd like |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
amazing 👍
Changes
Several github adjustments
Checklist