Use Gradle-native formatter #1208

Merged
mmarif merged 7 commits from qwerty287/GitNex:spotless into main 2 weeks ago
Collaborator

Run gradle :app:spotlessApply to apply formatting.

Changes from current code format

This does not match with the old intellij based code format because I can't apply some rules there.

  • newlines after else, catch etc. (means there are } else { statements)
  • no spaces around if braces (there are statements like if (true) { instead of if(true) {

Closes #1187
Closes #1202

Run `gradle :app:spotlessApply` to apply formatting. # Changes from current code format This does not match with the old intellij based code format because I can't apply some rules there. - newlines after `else`, `catch` etc. (means there are `} else {` statements) - no spaces around `if` braces (there are statements like `if (true) {` instead of `if(true) {` Closes https://codeberg.org/gitnex/GitNex/issues/1187 Closes https://codeberg.org/gitnex/GitNex/issues/1202
qwerty287 added 4 commits 3 weeks ago
c27abb4231
Update wp config
qwerty287 added the
🏗️ Build
label 3 weeks ago
qwerty287 self-assigned this 3 weeks ago
Owner

It seems that is the default style for java with google formatter. We can't do much I guess in that.

It seems that is the default style for java with google formatter. We can't do much I guess in that.
Poster
Collaborator

Yes, that's the issue I believe. I can/should change the IntelliJ code style.

Yes, that's the issue I believe. I can/should change the IntelliJ code style.
6543 approved these changes 3 weeks ago
6543 added this to the 5.0.0 milestone 3 weeks ago
6543 added 1 commit 3 weeks ago
a1cadf435f Merge branch 'main' into spotless
Collaborator

@mmarif I'm ready if you are

@mmarif I'm ready if you are
qwerty287 added 1 commit 3 weeks ago
8de5f90e6d
Update intellij config and change checking in ci
qwerty287 added 1 commit 2 weeks ago
0402cf19e5
Merge branch 'main' into spotless
mmarif approved these changes 2 weeks ago
mmarif referenced this issue from a commit 2 weeks ago
mmarif merged commit 325560c08d into main 2 weeks ago

Reviewers

6543 approved these changes 3 weeks ago
mmarif approved these changes 2 weeks ago
ci/woodpecker/pr/locale Pipeline was successful
ci/woodpecker/pr/check Pipeline was successful
ci/woodpecker/pr/build Pipeline was successful
The pull request has been merged as 325560c08d.
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: gitnex/GitNex#1208
Loading…
There is no content yet.