Skip to content

Fix CI.yml and also test on Julia 1.12 - #195

Merged
ufechner7 merged 2 commits into
mainfrom
fix_ci
Aug 14, 2025
Merged

Fix CI.yml and also test on Julia 1.12#195
ufechner7 merged 2 commits into
mainfrom
fix_ci

Conversation

@ufechner7

Copy link
Copy Markdown
Member

Fix some small issues.

@ufechner7
ufechner7 marked this pull request as draft August 13, 2025 19:35
@codecov

codecov Bot commented Aug 13, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@ufechner7 ufechner7 linked an issue Aug 13, 2025 that may be closed by this pull request
@ufechner7 ufechner7 changed the title Fix CI.yml Fix CI.yml and also test on Julia 1.12 Aug 14, 2025
@ufechner7
ufechner7 marked this pull request as ready for review August 14, 2025 07:10
@ufechner7
ufechner7 requested a review from Copilot August 14, 2025 07:10

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes issues in the GitHub Actions CI workflow and adds testing for Julia pre-release versions. The changes improve the workflow structure by reordering steps and removing duplicate PyCall build commands.

  • Adds "pre" Julia version to test matrix for pre-release testing
  • Removes duplicate PyCall build step that was running before Julia setup
  • Reorders CI steps to follow proper sequence (setup Julia, cache, build packages, then run tests)

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@ufechner7
ufechner7 requested a review from 1-Bart-1 August 14, 2025 07:37
@1-Bart-1

Copy link
Copy Markdown
Member

We can also change the matrix to
'1'
'lts'
'pre'

@ufechner7
ufechner7 merged commit 9a5a669 into main Aug 14, 2025
14 checks passed
1-Bart-1 pushed a commit that referenced this pull request Jul 1, 2026
* Fix CI.yml

* Also test on 1.12
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.

Something is wrong in CI.yml

3 participants