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

feat: add output for debugging CI #14

Merged
merged 7 commits into from
Nov 25, 2023
Merged

feat: add output for debugging CI #14

merged 7 commits into from
Nov 25, 2023

Conversation

SimeonC
Copy link
Contributor

@SimeonC SimeonC commented Oct 11, 2023

I needed to debug what the env vars were being set to inside our CI. This is often difficult to debug so adding an option to just dump the passed values out to the CI output would be useful to me.

CleanShot 2023-10-11 at 18 49 58

@Julien-R44
Copy link
Owner

Hey! Thanks for the PR and sorry for taking some time

I updated your PR with some things :

  • Prettier output
    image
  • Tests for this feature
  • Renamed showOutput to debug
  • Variables wasn't logged if validation was failing. So I also fixed this

Should be all good now

@Julien-R44 Julien-R44 merged commit f48f8bc into Julien-R44:main Nov 25, 2023
0 of 3 checks passed
@SimeonC
Copy link
Contributor Author

SimeonC commented Nov 27, 2023

Nice thanks for cleaning it up!

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.

None yet

2 participants