Skip to content

Adds builds to CI for pushes to main#3917

Merged
DrPaulSharp merged 2 commits intomainfrom
fix_ci
Apr 7, 2026
Merged

Adds builds to CI for pushes to main#3917
DrPaulSharp merged 2 commits intomainfrom
fix_ci

Conversation

@DrPaulSharp
Copy link
Copy Markdown
Contributor

Description

Modifies the CI to also create and test a build for pushes to main. We may want a more elegant approach to this, but this should fix our CI problems for now.

Fixes #3915

How Has This Been Tested?

Test requires a merge.

Review Checklist:

[if using the editor, use [x] in place of [ ] to check a box]

Documentation (check at least one)

  • There is nothing that needs documenting
  • Documentation changes are in this PR
  • There is an issue open for the documentation (link?)

Installers

  • There is a chance this will affect the installers, if so
    • Windows installer (GH artifact) has been tested (installed and worked)
    • MacOSX installer (GH artifact) has been tested (installed and worked)
    • Wheels installer (GH artifact) has been tested (installed and worked)

Licensing (untick if necessary)

  • The introduced changes comply with SasView license (BSD 3-Clause)

@DrPaulSharp DrPaulSharp requested a review from krzywon April 7, 2026 15:34
@DrPaulSharp
Copy link
Copy Markdown
Contributor Author

Closed in favour of #3916

@DrPaulSharp DrPaulSharp closed this Apr 7, 2026
@krzywon krzywon reopened this Apr 7, 2026
@DrPaulSharp
Copy link
Copy Markdown
Contributor Author

#3916 didn't work, so reopening here.

Copy link
Copy Markdown
Contributor

@krzywon krzywon left a comment

Choose a reason for hiding this comment

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

This should fix the issue, but the check might be slightly more specific than we need.

@DrPaulSharp DrPaulSharp merged commit 4eb8450 into main Apr 7, 2026
36 checks passed
@DrPaulSharp DrPaulSharp deleted the fix_ci branch April 7, 2026 16:37
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.

Nightly Builds failing

2 participants