Skip to content

Conversation

@JacobDomagala
Copy link
Member

Fixes #396

@JacobDomagala JacobDomagala self-assigned this Aug 4, 2025
@JacobDomagala JacobDomagala linked an issue Aug 4, 2025 that may be closed by this pull request
@codecov
Copy link

codecov bot commented Aug 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.37%. Comparing base (93b0c5b) to head (6ba6d23).
⚠️ Report is 5 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #397   +/-   ##
========================================
  Coverage    96.37%   96.37%           
========================================
  Files           89       89           
  Lines         2845     2845           
========================================
  Hits          2742     2742           
  Misses         103      103           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@JacobDomagala JacobDomagala force-pushed the 396-add-new-build-badges branch 2 times, most recently from 935ec1a to 791ba82 Compare August 4, 2025 12:23
@JacobDomagala
Copy link
Member Author

@JacobDomagala JacobDomagala marked this pull request as ready for review August 4, 2025 12:27
@JacobDomagala
Copy link
Member Author

JacobDomagala commented Aug 4, 2025

We need to merge DARMA-tasking/badge-generator#2 first ✔️

@JacobDomagala JacobDomagala force-pushed the 396-add-new-build-badges branch from 791ba82 to 0e8c772 Compare August 5, 2025 15:31
Copy link
Contributor

@lifflander lifflander left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@lifflander lifflander merged commit b5a9990 into develop Aug 5, 2025
19 checks passed
run: |
data=$(gh api /repos/DARMA-tasking/magistrate/actions/runs/$RUN_ID/jobs)

printf "names<<EOF\n%s\nEOF\n" "$(echo "$data" | jq -r '.jobs[] | select(.name | startswith("bake")) | .name | ltrimstr("bake (") | rtrimstr(")")')" >>"$GITHUB_OUTPUT"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@JacobDomagala Optional: we could trim magistrate-build- from the beginning (or maybe even magistrate-build-amd64-) to make the badges shorter and focused on the important aspects of the builds.
image

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.

Add new build badges

4 participants