Skip to content

Conversation

@matbesancon
Copy link
Member

This will probably break for now. I took the structure from https://github.com/ERGO-Code/HiGHS/blob/master/.github/workflows/build-windows.yml

@fschloesser
Copy link
Collaborator

fschloesser commented Apr 8, 2022

looks good to me so far, i think we could even try to add windows-latest for the os, but then maybe we have to take out the shell, since bash is the default anyways for unix and windows should run on powershell https://dev.to/pwd9000/github-actions-all-the-shells-581h

@matbesancon
Copy link
Member Author

ping @gasse, can you activate / allow workflows for this PR?

@matbesancon matbesancon reopened this May 10, 2022
@gasse
Copy link

gasse commented May 10, 2022

Hi @matbesancon , I have no idea how to activate / allow workflows :P Do you want me to merge this pull request? Maybe @AntoinePrv knows better what to do?

@matbesancon
Copy link
Member Author

it might require allowing actions to run, either in this PR if there is the option, or at the repo level yes

@AntoinePrv
Copy link
Collaborator

It seems to be enabled, but I don't see the message to allow running workflow for first time contributors...
IIRC Github Actions do not run the first time, I'll commit an empty workflow to the main branch.

@AntoinePrv
Copy link
Collaborator

The workflow is not valid. .github/workflows/ci.yml (Line: 15, Col: 9): A sequence was not expected

These Yaml configs are really a nightmare to debug...

@svigerske svigerske force-pushed the workflows branch 3 times, most recently from c508feb to 89504ca Compare December 10, 2025 17:19
@svigerske
Copy link
Member

There was still an issue with the strategy: matrix part: an include: was missing.
The workflow is executed now.

@svigerske svigerske force-pushed the workflows branch 2 times, most recently from 3c09c80 to df1a35f Compare December 10, 2025 17:46
@svigerske svigerske changed the title add unix workflow add workflow that builds bliss on different platforms for release or debug Dec 10, 2025
@svigerske
Copy link
Member

looks good to me so far, i think we could even try to add windows-latest for the os, but then maybe we have to take out the shell

yes, that seems to work fine

@svigerske
Copy link
Member

@fschloesser, I think this is finally ready for your review :)

#5 has many more ideas what one could do, but I think this PR is already enough good for now.

matbesancon and others added 3 commits December 11, 2025 10:26
- adding buildtype to matrix
- add runs on ubuntu-24.04-arm, intel-macos, windows-latest, and
  windows-arm runners
@svigerske svigerske force-pushed the workflows branch 2 times, most recently from fa9dfb8 to 09a8c07 Compare December 11, 2025 09:38
@svigerske svigerske merged commit 9aaecd2 into scipopt:master Dec 13, 2025
12 checks passed
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.

5 participants