-
-
Notifications
You must be signed in to change notification settings - Fork 2.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
checkstyle configuration for API #11859
base: main
Are you sure you want to change the base?
Conversation
Ok, this is ready to be reviewed. I'm sure there will be things we notice when we start updating all the files it tells us to that we don't want, but we can make those changes as we go along. This also isn't validated against any formatter yet, so there could be conflicts. We will find those out though as we go. It also isn't running the custom import order yet, because that will come with the formatter. |
10c4bb0
to
e015bb8
Compare
So there's a ton of new logic now in the api's build.gradle. This should probably be moved somewhere, idk if we wanna do the buildSrc thing, or make it part of pw, or what |
b6e5ec7
to
567e547
Compare
also only gets diff for commits that are on HEAD but not remote main
567e547
to
6d981e0
Compare
The checkstyle configuration for the API. The server's will come later and be mostly similar with some key differences (like javadocs and excluding decompiled sources).
Has a file that includes all packages in the API that need to be done. Once removed from that list, all files in it will be subject to checkstyle.
Checkstyle Categories