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

DOC: Add Pixi-managed build status badge to README file #4949

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jhlegarreta
Copy link
Member

@jhlegarreta jhlegarreta commented Nov 14, 2024

Add Pixi-managed build status badge to README file. A single badge is added since the OS configurations are part of a job matrix.

Add an additional column to distinguish Azure pipelines and GHA builds.

PR Checklist

@github-actions github-actions bot added the type:Documentation Documentation improvement or change label Nov 14, 2024
@jhlegarreta
Copy link
Member Author

Notes:

  • Cannot get a badge per matrix configuration.
  • Should we remove the GHA macOS ARM64 workfow given that the pixi-managed builds are testing it?

@jhlegarreta jhlegarreta force-pushed the AddPixiStatusBadgeToREADME branch 3 times, most recently from 2ad78e3 to 5eebf9b Compare November 14, 2024 01:43
@thewtex
Copy link
Member

thewtex commented Nov 14, 2024

@jhlegarreta thanks!

This is what I see:

image

However, currently, the Pixi GitHub Actions configuration only runs C++ build/test.

Since the Pixi build uses the conda-forge toolchain, we can keep the GHA macOS ARM64 to have more toolchain coverage.

Add `pixi`-managed build status badge to README file. A single badge is
added to the C++ columns since the OS configurations are part of a job
matrix.

Add an additional column to distinguish Azure pipelines and GHA builds.
@jhlegarreta
Copy link
Member Author

👍 thanks Matt, yes that was what I intended to display. Removed the badge from the Python column.

@hjmjohnson hjmjohnson self-assigned this Nov 14, 2024
@hjmjohnson hjmjohnson added this to the ITK 6.0 Beta 1 milestone Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:Documentation Documentation improvement or change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants