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

misc: print script logs upon failure #686

Merged
merged 1 commit into from
Oct 2, 2024
Merged

misc: print script logs upon failure #686

merged 1 commit into from
Oct 2, 2024

Conversation

Nashtare
Copy link
Collaborator

@Nashtare Nashtare commented Oct 1, 2024

We used to have ./script.sh | tee log.out for convenience to retrieve the errors arising in CI jobs, but this was quite cumbersome locally for quick testing runs, hence reverting to &> log.out, which obfuscates everything.

As a simple middle ground, we can print the logs only upon failure, to easily retrieve the failures on the CI while not clogging our terminals when testing

@Nashtare Nashtare added this to the x Misc. milestone Oct 1, 2024
@Nashtare Nashtare self-assigned this Oct 1, 2024
@Nashtare Nashtare requested a review from muursh as a code owner October 1, 2024 21:56
@Nashtare Nashtare merged commit ec5778d into develop Oct 2, 2024
20 checks passed
@Nashtare Nashtare deleted the misc/scripts branch October 2, 2024 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants