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

Set output for the error message #446

Merged
merged 3 commits into from
May 19, 2023

Conversation

wagnerm
Copy link
Contributor

@wagnerm wagnerm commented Apr 14, 2023

In the event vault-action throws an error, the error message is sent to the workflow log when with core.setFailed, but that output is not accessible to other steps in the workflow.

This change sets an output for the error message, called errorMessage, than can be read by other steps within the workflow and parsed.

In the event vault-action throws an error, the error message is
sent to the workflow log when with core.setFailed, but that output
is not accessible to other steps in the workflow.

This change sets an output for the error message, called errorMessage,
than can be read by other steps within the workflow and parsed.
@wagnerm wagnerm marked this pull request as ready for review April 14, 2023 17:39
@thyton thyton self-requested a review May 5, 2023 17:07
Copy link
Contributor

@thyton thyton left a comment

Choose a reason for hiding this comment

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

Hey @wagnerm, thanks for adding this good enhancement! Could we have it documented next to Features section under Unreleased in CHANGELOG.md?

Improvements:

* Set an output `errorMessage` to vault-action's error message so that following steps in the workflow can read [GH-446](https://github.com/hashicorp/vault-action/pull/446)

@thyton thyton added the waiting-for-response Indicates a maintainer asked a question that must be answered by contributors before proceeding label May 5, 2023
* main:
  Add userpass auth and ldap auth support (hashicorp#440)
  chore(deps-dev): bump jest from 29.4.3 to 29.5.0 (hashicorp#438)
@wagnerm wagnerm requested a review from thyton May 17, 2023 05:05
Copy link
Contributor

@maxcoulombe maxcoulombe left a comment

Choose a reason for hiding this comment

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

Thanks for adding this contribution and thinking about editing the unreleased changes!

@maxcoulombe maxcoulombe merged commit 1f5b7d5 into hashicorp:main May 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
waiting-for-response Indicates a maintainer asked a question that must be answered by contributors before proceeding
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants