-
-
Notifications
You must be signed in to change notification settings - Fork 3
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
[Formatting] Remove blank spaces #44
Conversation
Remove blank spaces for formatting Signed-off-by: akkd <[email protected]>
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.
Alright but next time use the actual PR template you hear?
There would be no applicable or reasonable use to apply the PR template on a change like mine. How would my formatting correction be used in the future? The PR template makes some sense for a larger addition that is making more changes, but for smaller changes, it is almost completly invalidated. |
I don't mean you keep the whole template. Just the general format and remove anything that doesn't work. It's just more uniform that way. |
The only thing I would have kept in that case would have been the changes made- Which I still feel would be a bit exessive to have Changes made
or something along those lines. Perhaps I could have done something along the line of Changes made
To make it less empty. |
Yeah, I don't actually have any issues with it, just for next time. |
Ex, For when there isnt that many things changed, there is less of a need for bulleting or listing items. |
Remove blank spaces for formatting reasons