Skip to content

Conversation

@ctubbsii
Copy link
Member

set-output was previously used to create the IT matrix, but it is now deprecated. So, this change makes use of the environment files instead.

See https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions#environment-files

@ctubbsii ctubbsii requested a review from EdColeman October 14, 2022 18:02
@ctubbsii ctubbsii self-assigned this Oct 14, 2022
set-output was previously used to create the IT matrix, but it is now
deprecated. So, this change makes use of the environment files instead.

See https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions#environment-files
@ctubbsii ctubbsii force-pushed the update-github-actions branch from 3ac3345 to 9908f6e Compare October 14, 2022 18:24
Copy link
Contributor

@EdColeman EdColeman left a comment

Choose a reason for hiding this comment

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

Changes LGTM - unless the results of the full test run revel any issues.

@ctubbsii
Copy link
Member Author

I'm troubleshooting an error that seems to be a data type issue:

Error when evaluating 'strategy' for job 'mvn'. .github/workflows/maven-full-its.yaml (Line: 88, Col: 15): Error parsing fromJson,.github/workflows/maven-full-its.yaml (Line: 88, Col: 15): Error reading JToken from JsonReader. Path '', line 0, position 0.,.github/workflows/maven-full-its.yaml (Line: 88, Col: 15): Unexpected type of value '', expected type: Mapping.

It looks like GitHub is parsing environment variables as String types, and I can't seem to convert it to a "Mapping" type.

@ctubbsii
Copy link
Member Author

Ignore my previous observations. I forgot to update the job output mapping to reflect the new step output name. I'm testing now, but I think the current bits will work.

@ctubbsii
Copy link
Member Author

Okay, got it working. Apparently, you're not supposed to quote the output value, but leave the raw JSON... which is weird, because it makes an invalid environment variable assignment in a shell script... but apparently that's what it wants.

@ctubbsii ctubbsii merged commit 04ac8b2 into apache:main Oct 14, 2022
@ctubbsii ctubbsii deleted the update-github-actions branch October 14, 2022 21:07
@ctubbsii ctubbsii added this to the 2.1.0 milestone Jul 12, 2024
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