Skip to content
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

Upgrade Gradle, Java & Dependencies #850

Merged
merged 12 commits into from
Aug 6, 2023

Conversation

Nicklas2751
Copy link
Member

Upgrade Gradle to 7.5
Upgrade Java to latest LTS 17
Upgrade dependencies
Remove useless nighlty action
Created new release action which publishes container & creates GitHub release simmilar to the new develop pre-release action

Copy link
Member

@TheSasch TheSasch left a comment

Choose a reason for hiding this comment

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

Only a few AssertJ Findings.

@sonarcloud
Copy link

sonarcloud bot commented Oct 22, 2022

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell C 5 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@TheSasch TheSasch self-requested a review October 22, 2022 19:24
@pidoubleyou
Copy link
Contributor

hoffentlich hat @alex1702 auch ein Java 17 auf den Crawler-Systemen ;)

@pidoubleyou
Copy link
Contributor

@Nicklas2751 @alex1702 was ist eigentlich mit diesem PR? ist dieser noch relevant?

@Nicklas2751
Copy link
Member Author

Aus meiner Sicht ja

@alex1702
Copy link
Member

alex1702 commented May 2, 2023

Aktuell hängt es bei mir grad noch am Buildsystem, da ich dort nur Java 8 und 9 zur verfügung habe. Es steht aber auf meiner ToDo es komplett umzubauen. Die Crawlersysteme habe ich schon mit Java 17 ausgestattet als Runtime.

@alex1702 alex1702 merged commit 1c8c81c into master Aug 6, 2023
0 of 2 checks passed
@alex1702 alex1702 deleted the feature/upgradeMasterDependencies branch August 8, 2023 07:45
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.

4 participants