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

CI log improvements #621

Merged
merged 79 commits into from
Nov 30, 2023
Merged

CI log improvements #621

merged 79 commits into from
Nov 30, 2023

Conversation

jrhemstad
Copy link
Collaborator

@jrhemstad jrhemstad commented Oct 25, 2023

Description

In an effort to make the CI logs more delightful, this PR does a few things:

  • Uses groups to collapse and group together logical sections
  • Uses color/emojis to better highlight important information
  • Adds a ninja_summary.py script to display the 10 longest build times and includes it in the CI log
  • Updated the repro instructions wording in the run-as-coder workflow to make it easier to understand

Example of a failed build log
latest-screenshot

Example of a successful build log
latest-screenshot

@jrhemstad jrhemstad requested review from a team as code owners October 25, 2023 22:29
@jrhemstad jrhemstad requested review from jarmak-nv and alliepiper and removed request for a team October 25, 2023 22:29
@jrhemstad jrhemstad requested a review from a team as a code owner October 26, 2023 20:17
@jrhemstad jrhemstad changed the title Experiment with adding groups to CI script steps CI log improvements Oct 31, 2023
@jrhemstad jrhemstad merged commit c4769d7 into main Nov 30, 2023
515 checks passed
@miscco miscco deleted the jrhemstad-patch-10 branch March 7, 2024 19:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

1 participant