Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add unix workflow #3

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

matbesancon
Copy link

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
Author

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

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

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
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
Member

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
Member

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...

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.

4 participants