-
-
Notifications
You must be signed in to change notification settings - Fork 354
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
Code standards update May 2024. #6684
base: dev/patch
Are you sure you want to change the base?
Conversation
Now might also be a good time to put the This will speed up PRs even further, as developers no longer need to manually adjust their IDE settings for Skript. Links:
|
Tud has already agreed to do this |
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.
Sounds good then
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.
just some thoughts :)
Co-authored-by: Patrick Miller <apickledwalrus@gmail.com>
Description
Our standards committee have decided upon a set of changes to code conventions and contribution standards, designed to make things a little clearer and more modern.
Some former rules have been relaxed (where we no longer felt the restrictions were helpful/necessary) and some new things have been standardised.
Overview of Changes
@NotNull
) can now go before the value, rather than the method (e.g.public @NotNull Object myMethod()
){
brackets}
around single-line blocks.if ... else {...}
)Target Minecraft Versions: any
Requirements: none
Related Issues: none