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

Modernization #2

Closed
wants to merge 5 commits into from
Closed

Modernization #2

wants to merge 5 commits into from

Conversation

peternied
Copy link

@peternied peternied commented Sep 27, 2023

Description

  • Fix broken name output parameter in README.md
  • Pulled in latest version of @action/core to mitigate security issue
  • Upgraded to node16
  • Added test case on push/pr

Issues

- Fix broken name output parameter in README.md
- Pulled in latest version of @action/core to mitigate security issue
- Upgraded to node16
- Added test case on push/pr

Signed-off-by: Peter Nied <[email protected]>
Signed-off-by: Peter Nied <[email protected]>
@peternied
Copy link
Author

@TGPSKI I'm going to close this out - if you do come across this PR feel free to reopen.

@peternied peternied closed this Oct 13, 2023
@TGPSKI
Copy link
Owner

TGPSKI commented Dec 18, 2023

Hey, just saw this. I was out on leave for a few months.

@peternied
Copy link
Author

Welcome back - what would you like to do with this PR? If you like the direction, I'm happy to revisit and then cut-over my project to this tool

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.

The set-env command is disabled.
2 participants