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

CI: remove Gradle caching in actions/setup-java to avoid warnings on Windows runners #1438

Merged
merged 2 commits into from
Feb 29, 2024

Conversation

tgrothe
Copy link
Contributor

@tgrothe tgrothe commented Feb 29, 2024

Es lag an dem Gradle-Cache (des Gradle-Plugins). Diesen habe ich in diesem PR entfernt.

closes #1354


It was due to the Gradle cache (of the Gradle plugin). I removed it in this pull request.

- gradle cache only once
@tgrothe tgrothe added bug Something isn't working tooling labels Feb 29, 2024
@tgrothe tgrothe self-assigned this Feb 29, 2024
@tgrothe tgrothe requested a review from cagix as a code owner February 29, 2024 06:24
@cagix
Copy link
Member

cagix commented Feb 29, 2024

Ich vermute, den Gradle-Cache brauchen wir nur einmal. Zudem sollte einmal (anfangs) clean aufgerufen werden.

"Vermuten" hört sich in einem PR nicht vertrauenswürdig an.

Und build basiert auf check, und alle Nachfolgenden basieren auf build.

Check prüft nur, ob an den JUnit-Sourcen selbst Änderungen vorgenommen wurden. Insofern ich Check komplett unabhängig. Die JUnit-Jobs hängen dagegen von Build und entsprechend von Check ab.

Spotless mag sein, dass es nur nach einem Build sinnvoll ist, weil es sonst eh nicht läuft.

Edit: Funktioniert leider nicht wie gedacht. Der tar Fehler besteht weiterhin.

Dann gewöhne Dir doch endlich mal an, das verd**mte Ding auf "Draft" zu stellen. Warum soll ich ein Review machen, wenn es eh nicht geht?!

@cagix cagix marked this pull request as draft February 29, 2024 07:39
.github/workflows/github_ci.yml Outdated Show resolved Hide resolved
- restore formats
- remove "cache: gradle"
Copy link
Member

@cagix cagix left a comment

Choose a reason for hiding this comment

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

lgtm, thx

@cagix cagix changed the title CI: Fix warnings CI: remove Gradle caching in actions/setup-java to avoid warnings on Windows runners Feb 29, 2024
@cagix
Copy link
Member

cagix commented Feb 29, 2024

@tgrothe Danke, das sollte so gehen.

Das mit dem "Draft" hast Du aber immer noch nicht verstanden, oder? So lange Du an einem PR arbeitest, setzt Du den bitte auf "Draft", damit ich nicht bei jedem Pups eine Benachrichtigung bekomme. Wenn Du fertig bist und zum Review aufrufst, dann nimmst Du den "Draft"-Status wieder raus.

@cagix cagix marked this pull request as ready for review February 29, 2024 16:48
@cagix cagix merged commit 702509e into master Feb 29, 2024
7 checks passed
@cagix cagix deleted the tg/fix_java_ci branch February 29, 2024 16:49
@tgrothe
Copy link
Contributor Author

tgrothe commented Feb 29, 2024

@cagix alles klar:)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working tooling
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI: Warning in Windows runner
2 participants