-
-
Notifications
You must be signed in to change notification settings - Fork 169
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
docs: merged coding_style_standards.md with #820 #852
docs: merged coding_style_standards.md with #820 #852
Conversation
Co-authored-by: Jakub 'Eremiell' Marek <eremiell@eremiell.net>
✅ Deploy Preview for dpp-dev ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
What's the purpose of this PR? these changes are the same as in #820 |
The current coding style standards have a slight issue with them (contradicting itself). Rather than just pushing a fix for that, I thought it would be ideal to just drag over the change in #820 into here as that already has the fix but also expands upon a couple sections. |
Requested Review again from you @Commandserver as I've made another change, adding a new section. Don't want you to miss it in-case you don't like something about it. |
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.
Looks good :D
…oxdotcc#852) Co-authored-by: Jakub 'Eremiell' Marek <eremiell@eremiell.net>
This PR merges
coding_style_standards.md
with #820. This is because, previously, the coding standards were slightly incorrect and needed fixing (as requested by Brain).dev
branch.