Skip to content

Conversation

@cmitzel-ncino
Copy link
Contributor

@cmitzel-ncino cmitzel-ncino commented Apr 3, 2023

WIP, trying to implement a flag to the action such that the action will result in a success status regardless of errors or warnings.

I need to implement one or two tests still, I'm pretty new to developing in TS outside of the AWS CDK so taking my time to learn.

@cmitzel-ncino
Copy link
Contributor Author

cmitzel-ncino commented Apr 5, 2023

linked with issue #687

@cmitzel-ncino
Copy link
Contributor Author

@wagoid I added a couple tests and some docs, let me know if this needs anything else!

Copy link
Owner

@wagoid wagoid left a comment

Choose a reason for hiding this comment

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

Awesome work @cmitzel-ncino! Could you also add this param to action.yml file? Like here

@wagoid
Copy link
Owner

wagoid commented Apr 10, 2023

Merging it, thanks @cmitzel-ncino!

@wagoid wagoid merged commit 3b6f002 into wagoid:master Apr 10, 2023
@cmitzel-ncino
Copy link
Contributor Author

Hitting an issue with the action reporting

Error: Unable to process file command 'output' successfully.
Error: Invalid format 'You have commit messages with errors

Looking at the docs I believe I'm missing an argument for the setOutput() function so I'm rectifying that now.

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