Skip to content

Conversation

@tylervz
Copy link
Contributor

@tylervz tylervz commented Jun 21, 2024

Update the GitHub actions workflows to use the latest version of multiple actions, including actions/checkout, actions/setup-java, actions/cache, and release-drafter/release-drafter:

https://github.com/actions/checkout/tags
https://github.com/actions/setup-java/tags
https://github.com/actions/cache?tab=readme-ov-file#v4
https://github.com/release-drafter/release-drafter/tags

- name: Run Build
id: build
uses: gradle/gradle-build-action@v2
run: ./gradlew build
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The gradle/actions/setup-gradle action replaces gradle/gradle-build-action.
Reference:
https://github.com/gradle/actions/tree/af6e5767241073b8ab20cc8cf7e95ec63fadc160/setup-gradle#the-setup-gradle-action

run: echo "value=${GITHUB_REF:11}" >> $GITHUB_OUTPUT
# If it has release drafter:
- uses: release-drafter/release-drafter@v5
- uses: release-drafter/release-drafter@v6
Copy link
Contributor Author

Choose a reason for hiding this comment

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

There are no breaking changes in this release:
https://github.com/release-drafter/release-drafter/releases/tag/v6.0.0

@tylervz
Copy link
Contributor Author

tylervz commented Jun 21, 2024

I did not upgrade the "Build and install groovy (no docs)" step in groovy-joint-workflow.yml, because I did not see a suitable replacement using gradle/actions/setup-gradle@v3.

If I replaced it with gradle/actions/setup-gradle@v3 like so:

- name: Setup Gradle
uses: gradle/actions/setup-gradle@v3
with:
  build-scan-publish: true
  build-scan-terms-of-use-url: "https://gradle.com/terms-of-service"
  build-scan-terms-of-use-agree: "yes"
# Use the Gradle wrapper in the ../groovy directory
- name: Build and install groovy (no docs)
  run: |
    cd ../groovy
    ./gradlew install \
      -x groovydoc \
      -x javadoc \
      -x javadocAll \
      -x groovydocAll \
      -x asciidoc \
      -x docGDK
  env:
    GRADLE_SCANS_ACCEPT: yes
    GRADLE_ENTERPRISE_ACCESS_KEY: ${{ secrets.GRADLE_ENTERPRISE_ACCESS_KEY }}

Then the gradle/actions/setup-gradle@v3 step in the example above is useless since it is configuring Gradle in the root directory for optimal execution rather than the Gradle wrapper in ../groovy that is actually going to get used.

I do not see any inputs/parameters for gradle/actions/setup-gradle@v3 similar to build-root-directory to have it configure a Gradle wrapper in a directory other than the root directory.
https://github.com/gradle/actions/blob/main/setup-gradle/action.yml

Update: perhaps we should update the step to be the same as it is in grails-core?
https://github.com/grails/grails-core/blob/6.2.x/.github/workflows/groovy-joint-workflow.yml#L103-L119

jamesfredley and others added 22 commits August 17, 2024 18:20
…ations (apache#1810)

* javax to jakarta, groovy 4, spring framework 6 and spring boot 3 migrations
---------

Co-authored-by: Mattias Reichel <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: puneetbehl <[email protected]>
Co-authored-by: Puneet Behl <[email protected]>
Co-authored-by: Guillermo Calvo <[email protected]>
Co-authored-by: jhron <[email protected]>
Co-authored-by: micronaut-build <[email protected]>
Co-authored-by: Guillermo Calvo <[email protected]>
…pache#1817)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…1.9.0 (apache#1805)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
fix(deps): update dependency com.h2database:h2 to v2.3.232
jdaugherty and others added 23 commits October 29, 2024 20:52
Cleanup & fix annotation reading for classpath scanning
Revert classpath scanning change, use the same strategy as grails-core
Remove api dependency until groovy version stabilizes
remove groovy transitive dependency until groovy version stabilizes
Updates since groovy 4.0.24 is released
Add back logic for groovy snapshot support
@jamesfredley
Copy link
Contributor

@tylervz after #1849 has been merged, can you rebase this against 9.0.x?

@tylervz
Copy link
Contributor Author

tylervz commented Nov 13, 2024

@jamesfredley yes, I can do that :)

matrei and others added 2 commits November 13, 2024 21:56
Also update to develocity gradle plugin and related change to DEVELOCITY_* env vars.
@tylervz
Copy link
Contributor Author

tylervz commented Nov 13, 2024

I've rebased my branch on 9.0.x. I'm going to close this pull request and open a new one requesting to be merged into 9.0.x.

@tylervz tylervz closed this Nov 13, 2024
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.

8 participants