Skip to content

Commit

Permalink
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Compile with -parameters flag and upgrade GitHub Actions
Browse files Browse the repository at this point in the history
Compile hermes with `-parameters` flag, because newer Spring drops parameter resolution through bytecode scanning, see https://github.com/spring-projects/spring-framework/wiki/Upgrading-to-Spring-Framework-6.x#parameter-name-retention
deejay1 committed Mar 7, 2024

Verified

This commit was signed with the committer’s verified signature.
1 parent 8c9e700 commit 061996a
Showing 5 changed files with 6 additions and 5 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/checkstyle.yml
Original file line number Diff line number Diff line change
@@ -22,7 +22,7 @@ jobs:
java-version: 17
distribution: 'temurin'
- name: Setup Gradle
uses: gradle/gradle-build-action@v2
uses: gradle/actions/setup-gradle@v3
- name: Run check style
# ignore lengthy console setup tasks
run: ./gradlew --continue clean checkstyleMain checkstyleTest checkstyleIntegrationTest checkstyleSlowIntegrationTest checkstyleJmh -PmaxCheckstyleWarnings=0 -x attachHermesConsole -x prepareIndexTemplate
2 changes: 1 addition & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
@@ -29,7 +29,7 @@ jobs:
java-version: 17
distribution: 'temurin'
- name: Setup Gradle
uses: gradle/gradle-build-action@v2
uses: gradle/actions/setup-gradle@v3
- name: Build with Gradle
run: ./gradlew assemble
- name: Run task with Gradle
2 changes: 1 addition & 1 deletion .github/workflows/gradle-wrapper-validation.yml
Original file line number Diff line number Diff line change
@@ -13,4 +13,4 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: gradle/wrapper-validation-action@v1
- uses: gradle/wrapper-validation-action@v2
2 changes: 1 addition & 1 deletion .github/workflows/release.yml
Original file line number Diff line number Diff line change
@@ -25,7 +25,7 @@ jobs:
java-version: '17'
distribution: 'temurin'
- name: Setup Gradle
uses: gradle/gradle-build-action@v2
uses: gradle/actions/setup-gradle@v3
- name: Release
if: github.ref == 'refs/heads/master'
run: ./gradlew release -Prelease.customPassword=${GITHUB_TOKEN} -Prelease.customUsername=${GITHUB_ACTOR} -Prelease.forceVersion=${FORCE_VERSION}
3 changes: 2 additions & 1 deletion build.gradle
Original file line number Diff line number Diff line change
@@ -8,7 +8,7 @@ buildscript {

plugins {
id 'java'
id 'pl.allegro.tech.build.axion-release' version '1.13.6'
id 'pl.allegro.tech.build.axion-release' version '1.17.0'
id 'io.github.gradle-nexus.publish-plugin' version '1.0.0'
}

@@ -180,6 +180,7 @@ subprojects {

tasks.withType(JavaCompile) {
options.compilerArgs << "-Xlint:unchecked,deprecation"
options.compilerArgs << "-parameters"
}

test {

0 comments on commit 061996a

Please sign in to comment.