Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Update gradle#3392

Merged
Donnnno merged 15 commits into
mainfrom
updates
May 3, 2026
Merged

Update gradle#3392
Donnnno merged 15 commits into
mainfrom
updates

Conversation

@Donnnno
Copy link
Copy Markdown
Member

@Donnnno Donnnno commented May 3, 2026

From the codeQL action

@Donnnno Donnnno requested a review from Kaiserdragon2 May 3, 2026 11:33
@Kaiserdragon2
Copy link
Copy Markdown
Collaborator

Kaiserdragon2 commented May 3, 2026

I would leave the scripts as they are. They probably will brake with these changes. Also i think all of these aren't used anymore.

Copy link
Copy Markdown
Collaborator

@Kaiserdragon2 Kaiserdragon2 left a comment

Choose a reason for hiding this comment

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

Leave script's as they are. Gradle and manifest changes are fine.

@Donnnno
Copy link
Copy Markdown
Member Author

Donnnno commented May 3, 2026

Thanks, will do! Maybe we should remove them if we're not using them anyways?

Donnnno added 2 commits May 3, 2026 13:49
@Donnnno Donnnno changed the title Security fixes? Update gradle May 3, 2026
@Donnnno Donnnno temporarily deployed to github-pages May 3, 2026 11:58 — with GitHub Actions Inactive
@Donnnno Donnnno temporarily deployed to github-pages May 3, 2026 12:01 — with GitHub Actions Inactive
@Donnnno Donnnno temporarily deployed to github-pages May 3, 2026 12:03 — with GitHub Actions Inactive
@Donnnno Donnnno temporarily deployed to github-pages May 3, 2026 12:15 — with GitHub Actions Inactive
@Kaiserdragon2
Copy link
Copy Markdown
Collaborator

I always wanted to clean the scripts up a bit but never came to it. Yes indeed we should remove the old ones.

@Donnnno Donnnno temporarily deployed to github-pages May 3, 2026 15:51 — with GitHub Actions Inactive
@Donnnno Donnnno temporarily deployed to github-pages May 3, 2026 19:24 — with GitHub Actions Inactive
@Donnnno Donnnno merged commit 914e3eb into main May 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants