Skip to content

CI: Add verification CI job#2637

Merged
VeckoTheGecko merged 2 commits into
Parcels-code:mainfrom
VeckoTheGecko:push-swwtmqkwtozq
May 20, 2026
Merged

CI: Add verification CI job#2637
VeckoTheGecko merged 2 commits into
Parcels-code:mainfrom
VeckoTheGecko:push-swwtmqkwtozq

Conversation

@VeckoTheGecko
Copy link
Copy Markdown
Contributor

Description

xref #2633 (comment)

Checklist

AI Disclosure

  • This PR contains AI-generated content.
    • I have tested any AI-generated content in my PR.
    • I take responsibility for any AI-generated content in my PR.
    • Describe how you used it (e.g., by pasting your prompt): Asked claude to create an additional workflow mimicking the flaky one and using the Pixi task.

@VeckoTheGecko
Copy link
Copy Markdown
Contributor Author

Green ticks on CI - keen to know what you think @fluidnumericsJoe and @erikvansebille

This will become a "required check" in the GitHub CI

Copy link
Copy Markdown
Member

@erikvansebille erikvansebille 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 to me!

Copy link
Copy Markdown
Contributor

@fluidnumericsJoe fluidnumericsJoe left a comment

Choose a reason for hiding this comment

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

This looks good. I like the idea of separating out the validation tests from the other types of tests. Thinking about future triage, seeing failure in validation tests among other pass/fail items is a useful symptom that helps assist future developers in determining where an issue might be.

@VeckoTheGecko VeckoTheGecko merged commit 8730fdc into Parcels-code:main May 20, 2026
2 of 3 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in Parcels development May 20, 2026
@VeckoTheGecko VeckoTheGecko deleted the push-swwtmqkwtozq branch May 20, 2026 11:32
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