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.

Sign up for GitHub

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

Jump to bottom

Log all args in CI tests and on dev machines. #3442

Merged
merged 1 commit into from
Aug 12, 2024
Merged

Log all args in CI tests and on dev machines. #3442

merged 1 commit into from
Aug 12, 2024

Conversation

mitchell-as
Copy link
Contributor

@mitchell-as mitchell-as commented Aug 12, 2024 •
edited by github-actions bot
Loading

TaskDX-2983 Consider how to easily log commands with arguments, but filter out sensitive stuff

@mitchell-as
Copy link
Contributor Author

Test failures are not introduced by this PR, and prove that we log all flags and args.

@mitchell-as mitchell-as requested a review from MDrakos August 12, 2024 15:02
@mitchell-as mitchell-as marked this pull request as ready for review August 12, 2024 15:02
@mitchell-as mitchell-as reopened this Aug 12, 2024
@mitchell-as mitchell-as merged commit 6ff1b5a into version/0-46-0-RC1 Aug 12, 2024
13 of 20 checks passed
@mitchell-as mitchell-as deleted the mitchell/dx-2983 branch August 12, 2024 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MDrakos MDrakos MDrakos approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mitchell-as @MDrakos