Skip to content

chore(build): update - #2056

Merged
alperozturk96 merged 9 commits into
masterfrom
update-build-settings
Jul 22, 2026
Merged

chore(build): update#2056
alperozturk96 merged 9 commits into
masterfrom
update-build-settings

Conversation

@alperozturk96

@alperozturk96 alperozturk96 commented Jun 19, 2026

Copy link
Copy Markdown
Collaborator

Changes

  • Uses built-in kotlin
  • Removes deprecated gradle properties
  • Reduces ram usage
  • Enabled parallel sync for Gradle 9.4+

@github-actions

Copy link
Copy Markdown
Contributor

SpotBugs

CategoryBaseNew
Bad practice3131
Correctness2929
Dodgy code1414
Internationalization66
Malicious code vulnerability2626
Multithreaded correctness77
Performance77
Total120120

@alperozturk96
alperozturk96 force-pushed the update-build-settings branch from 5133c86 to 67215ed Compare June 23, 2026 06:14
@github-actions

Copy link
Copy Markdown
Contributor

SpotBugs

CategoryBaseNew
Bad practice3131
Correctness2828
Dodgy code1414
Internationalization66
Malicious code vulnerability2626
Multithreaded correctness77
Performance77
Total119119

@alperozturk96
alperozturk96 force-pushed the update-build-settings branch from 67215ed to a998b9c Compare July 6, 2026 11:18
@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

SpotBugs

CategoryBaseNew
Bad practice3131
Correctness2828
Dodgy code1414
Internationalization60
Malicious code vulnerability2626
Multithreaded correctness77
Performance77
Security44
Total123117

@alperozturk96
alperozturk96 force-pushed the update-build-settings branch from a998b9c to d44186f Compare July 16, 2026 11:35
@github-actions

Copy link
Copy Markdown
Contributor

SpotBugs

CategoryBaseNew
Bad practice3131
Correctness2828
Dodgy code1414
Internationalization60
Malicious code vulnerability2626
Multithreaded correctness77
Performance77
Security44
Total123117

@alperozturk96
alperozturk96 force-pushed the update-build-settings branch from d44186f to ed4a984 Compare July 16, 2026 13:59
@nextcloud-android-bot

Copy link
Copy Markdown
Collaborator

@github-actions

Copy link
Copy Markdown
Contributor

SpotBugs

CategoryBaseNew
Bad practice3131
Correctness2828
Dodgy code1414
Internationalization60
Malicious code vulnerability2626
Multithreaded correctness77
Performance77
Security44
Total123117

@alperozturk96
alperozturk96 force-pushed the update-build-settings branch from cc8ddcd to b97f64f Compare July 17, 2026 08:05
@github-actions

Copy link
Copy Markdown
Contributor

SpotBugs

CategoryBaseNew
Bad practice3131
Correctness2828
Dodgy code1414
Internationalization60
Malicious code vulnerability2626
Multithreaded correctness77
Performance77
Security44
Total123117

@alperozturk96

Copy link
Copy Markdown
Collaborator Author

only fails due to code ql kotlin version:

Screenshot 2026-07-17 at 11 16 43

Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>

# Conflicts:
#	.github/workflows/assemble.yml
#	.github/workflows/check.yml
#	.github/workflows/garm.yml
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>

# Conflicts:
#	.github/workflows/assemble.yml
#	.github/workflows/check.yml
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
@alperozturk96
alperozturk96 force-pushed the update-build-settings branch from b97f64f to 5d0d4d5 Compare July 21, 2026 07:01
@github-actions

Copy link
Copy Markdown
Contributor

SpotBugs

CategoryBaseNew
Bad practice3131
Correctness2828
Dodgy code1414
Internationalization60
Malicious code vulnerability2626
Multithreaded correctness77
Performance77
Security44
Total123117

Comment thread library/build.gradle Outdated
sourceCompatibility JavaVersion.VERSION_17
targetCompatibility JavaVersion.VERSION_17
sourceCompatibility JavaVersion.VERSION_21
targetCompatibility JavaVersion.VERSION_21

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can this be a problem for any app using our code?
If they are on 17?

@alperozturk96 alperozturk96 Jul 21, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sourceCompatibility = what language we write
targetCompatibility = what language level we produce -> this could break the clients

when i only set the targetCompatibility to 17 ci fails. thus better to keep 17.

Screenshot 2026-07-21 at 13 35 36

Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
@github-actions

Copy link
Copy Markdown
Contributor

SpotBugs

CategoryBaseNew
Bad practice3131
Correctness2828
Dodgy code1414
Internationalization66
Malicious code vulnerability2626
Multithreaded correctness77
Performance77
Security44
Total123123

@alperozturk96
alperozturk96 merged commit 20d4055 into master Jul 22, 2026
18 of 19 checks passed
@alperozturk96
alperozturk96 deleted the update-build-settings branch July 22, 2026 07:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants