Skip to content

Provide an additional 'message-file' as action output#57

Open
GGoetzelmann wants to merge 1 commit intomainfrom
provide_messagefile
Open

Provide an additional 'message-file' as action output#57
GGoetzelmann wants to merge 1 commit intomainfrom
provide_messagefile

Conversation

@GGoetzelmann
Copy link
Copy Markdown

Currently the message of the json check is written to a message string.

This works for showing the results in a job summary page, but it may pose problems for downstream workflow steps to further work with the message content, such as

  • very long content may not be usable as input for other actions (see for example Error Argument list too long with very large comment bodies peter-evans/create-or-update-comment#234)
  • message usually contains something like ```diff ...```, which might break all downstream steps that try to put the message content into a file via shell script (because the content in backticks might be interpreted as a command)
  • messages may contain other troublesome characters (such as single quotes etc) which makes the last point even more difficult to address.

To avoid such issues with the message string, this PR proposes to additionally write the message to an md file available for further workflow steps.

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.

1 participant