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

Use new gha API for setting output, latest core actions #167

Merged
merged 2 commits into from
Oct 25, 2022

Conversation

simonbasle
Copy link
Member

@simonbasle simonbasle commented Oct 18, 2022

This commit changes the way job output is defined, using the new way of
a GITHUB_OUTPUT file environment variable.

The old way of echoing strings is deprecated for sensitive elements,
including action/job output.

It also reorders the output in order to:

• ensure BAD version is detected before any output and fails the job
• ensure only one version/versionType is outputted when there are
multiple modules (here, the reactor-pool one)

Finally it updates core and gradle actions to latest versions.
This fixes warnings about deprecated output style and deprecated Node
version being still in use by these actions.

See reactor/reactor#727.

This commit changes the way job output is defined, using the new way of
a GITHUB_OUTPUT file environment variable.

The old way of echoing strings is deprecated for sensitive elements,
including action/job output.

It also reorders the output in order to:

• ensure BAD version is detected before any output and fails the job
• ensure only one version/versionType is outputted when there are
multiple modules (here, the `reactor-pool` one)

See also reactor/reactor-core#3237.
@simonbasle simonbasle added the type/chore A task not related to code (build, formatting, process, ...) label Oct 18, 2022
@simonbasle simonbasle self-assigned this Oct 18, 2022
@simonbasle simonbasle requested a review from a team October 18, 2022 14:45
GitHub core actions updated to v3 to also remove NodeJs deprecation
warnings:
 - checkout to v3.1.0
 - setup-java to v3.6.0

Also upgrade gradle/gradle-build-action to v2.3.3, which should remove
all the warnings about deprecated APIs.
@simonbasle simonbasle changed the title Use new gha API for setting output Use new gha API for setting output, latest core actions Oct 25, 2022
@simonbasle simonbasle merged commit c52e588 into 0.2.x Oct 25, 2022
@reactorbot
Copy link

@simonbasle this PR seems to have been merged on a maintenance branch, please ensure the change is merge-forwarded to intermediate maintenance branches and up to main 🙇

@simonbasle simonbasle deleted the actionsOutputNewApi branch October 25, 2022 16:55
simonbasle added a commit that referenced this pull request Oct 25, 2022
(fixing conflicts in the way only main module is taken into account)
@simonbasle simonbasle modified the milestones: 1.0 Planning, 0.2.11 Oct 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/chore A task not related to code (build, formatting, process, ...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants