Skip to content

[ci] Update CI workflow to include main/master branch and check label 'pr: run ci' in PR#8

Merged
th-skam merged 2 commits intomain-freefem-sofafrom
epernod-patch-1
Apr 3, 2026
Merged

[ci] Update CI workflow to include main/master branch and check label 'pr: run ci' in PR#8
th-skam merged 2 commits intomain-freefem-sofafrom
epernod-patch-1

Conversation

@epernod
Copy link
Copy Markdown
Collaborator

@epernod epernod commented Apr 2, 2026

No description provided.

Copy link
Copy Markdown
Collaborator

@th-skam th-skam left a comment

Choose a reason for hiding this comment

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

I think this PR should be based on main-freefem-sofa branch. That's where we would want to mute the CI with upcoming changes, unrelated to C++ dev.

For other stuff, I'll be making PRs in Alex's repository. He has his own workflow on GitHub.

epernod added 2 commits April 2, 2026 21:44
Updated CI workflow to trigger on push to main and master branches.
@epernod epernod changed the base branch from main to main-freefem-sofa April 2, 2026 19:46
Copy link
Copy Markdown
Collaborator Author

@epernod epernod left a comment

Choose a reason for hiding this comment

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

Done, I rebased it and request to merge into main-freefem-sofa

Copy link
Copy Markdown
Collaborator

@th-skam th-skam left a comment

Choose a reason for hiding this comment

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

So partial keyword main in main-freefem-sofa triggers the tests. 👍

@th-skam th-skam merged commit 44d22dc into main-freefem-sofa Apr 3, 2026
9 of 12 checks passed
@th-skam th-skam deleted the epernod-patch-1 branch April 3, 2026 07:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants