-
Notifications
You must be signed in to change notification settings - Fork 592
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
chore(ci): remove debug-only hmarr/debug-action
#14275
Conversation
Thanks for opening a PR! 💯
Howto
More infoPlease take a moment to read through the Magma project's
If this is your first Magma PR, also consider reading
|
hmarr/debug-action
version to remove deprecation wa…hmarr/debug-action
version to remove deprecation warning
@@ -26,7 +26,7 @@ jobs: | |||
WORKFLOW_NAME: "${{ github.event.workflow.name }}" | |||
WORKFLOW_STATUS: "${{ github.event.workflow_run.conclusion }}" | |||
steps: | |||
- uses: hmarr/debug-action@1201a20fc9d278ddddd5f0f46922d06513892491 # pin@v2 | |||
- uses: hmarr/debug-action@a701ed95a46e6f2fb0df25e1a558c16356fae35a # pin@v2.1.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems to be an action only used for debugging. It should thus probably not be in production code. If this is really the only occurrence, could we just remove it entirely instead please?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the only occurrence in the Magma repo and has been in this file since its original commit. The action only prints the environment variables and therefore may be removed in production code.
Signed-off-by: Marco Pfirrmann <marco.pfirrmann@tngtech.com>
1a20f7d
to
87f9b9f
Compare
hmarr/debug-action
version to remove deprecation warninghmarr/debug-action
Signed-off-by: Marco Pfirrmann <marco.pfirrmann@tngtech.com> Signed-off-by: Marco Pfirrmann <marco.pfirrmann@tngtech.com>
Summary
hmarr/debug-action
is a debug-only action and not needed in production code. Removed.Test Plan
Additional Information