Skip to content
This repository was archived by the owner on Dec 19, 2023. It is now read-only.

Update java 16 to 17 for PR test run, change distribution to temurin, fix publish unit test result task not working from forks #737

Conversation

BlasiusSecundus
Copy link

Eclipse Temurin is the successor of AdoptOpenJDK.

Eclipse Temurin is the successor of AdoptOpenJDK.
to investigate possible breaking change
to find where was the breaking change
to find where was the breaking change
@BlasiusSecundus BlasiusSecundus force-pushed the feature/ci-java-17-and-temurin branch from 31d7139 to 32eba98 Compare October 18, 2021 18:45
the breaking change appears to be introduced in 1.23
@BlasiusSecundus BlasiusSecundus changed the title ci: update java 16 to 17 for PR test run, change distribution to temurin / from feature branch / Update java 16 to 17 for PR test run, change distribution to temurin, fix publish unit test result task not working from forks Oct 18, 2021
@BlasiusSecundus
Copy link
Author

The issue is possibly related to this commit:
EnricoMi/publish-unit-test-result-action@e33a080

Although the documentation says (and from the code it appears) that only the event_file setting should enable running this action on a fork PRs, it seems that this behavior is present even if that setting is not present. Also, removing the files setting has no effect.

@BlasiusSecundus
Copy link
Author

Opened new PR #756 with just the Java update. The issue causing the builds to fail was fixed in dependency.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant