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

Add a github action to auto update the gradle wrapper #7651

Open
wants to merge 1 commit into
base: dev/feature
Choose a base branch
from

Conversation

TheLimeGlass
Copy link
Contributor

@TheLimeGlass TheLimeGlass commented Feb 26, 2025

Description

Adds an action to automatically update the Gradle wrapper to the latest version assuming it passes successfully.

See the GitHub bot commiting the changes at my branch https://github.com/TheLimeGlass/Skript/tree/update/gradle-example
and eventually here when the schedule is triggered which is set to weekly.

Task breakdown:

  1. Gets latest public Gradle release then collects the JSON result saving the version and download url
  2. Applies the version to the gradle.properties
  3. Runs the existing gradlew wrapper and tells it to update to the version in the gradle.properties
  4. After the wrapper is updated, we run a wrapper validation to verify that the wrapper is true
  5. Run a Skript quickTest
  6. Commits the changes in the gradle/wrapper folder using the official GitHub actions bot if none of the tasks failed.

Gradle updated to version 13 today, so it's the best time to update.


Target Minecraft Versions: any
Requirements: none
Related Issues: none

@Pikachu920
Copy link
Member

I like the idea, but how likely are we to encounter breaking changes? Can we attempt to build skript and not autoupdate if it fails?

@Pikachu920
Copy link
Member

Pikachu920 commented Feb 27, 2025

ooh I think it does that already. awesome

can we use this too? https://github.com/marketplace/actions/verified-bot-commit

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