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

fix(release-controller): remove suppression of stdout and stderr in Git commands #1172

Merged
merged 1 commit into from
Jan 3, 2025

Conversation

sasa-tomic
Copy link
Member

The changes in this PR involve the removal of the suppression of stdout and stderr for various Git commands in the GitRepo class and the host-os-release-notes.py script. This includes commands such as checkout, fetch, and reset, which now will output their standard messages and errors to the console.

Impact:

  • Improved visibility of Git command outputs, aiding in debugging and understanding of the operations being performed by the application.
  • Reduced potential difficulties in diagnosing issues, as the immediate feedback from Git operations is now available.

The changes in this PR involve the removal of the suppression of `stdout` and `stderr` for various Git commands in the `GitRepo` class and the `host-os-release-notes.py` script. This includes commands such as `checkout`, `fetch`, and `reset`, which now will output their standard messages and errors to the console.

Impact:
- Improved visibility of Git command outputs, aiding in debugging and understanding of the operations being performed by the application.
- Reduced potential difficulties in diagnosing issues, as the immediate feedback from Git operations is now available.
@sasa-tomic sasa-tomic requested a review from a team as a code owner January 3, 2025 11:11
@sasa-tomic sasa-tomic changed the title refactor: remove suppression of stdout and stderr in Git commands fix(release-controller): remove suppression of stdout and stderr in Git commands Jan 3, 2025
@sasa-tomic sasa-tomic enabled auto-merge (squash) January 3, 2025 11:37
@sasa-tomic sasa-tomic merged commit 63753dc into main Jan 3, 2025
5 checks passed
@sasa-tomic sasa-tomic deleted the refactor/git-repo-suppress-stdout branch January 3, 2025 11:49
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