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

Adds Github Actions workflow for test. #104

Merged
merged 2 commits into from
Mar 22, 2023
Merged

Conversation

arokem
Copy link
Collaborator

@arokem arokem commented Mar 14, 2023

Change to gitignore to ignore Mac OS Finder configuration files.

Unfortunately, I don't think that the workflow will run until this PR is merged.

@arokem arokem mentioned this pull request Mar 14, 2023
@arokem
Copy link
Collaborator Author

arokem commented Mar 14, 2023

I'll run this locally using act and report back here.

@arokem
Copy link
Collaborator Author

arokem commented Mar 15, 2023

Scratch that. I was not able to run the actions using act, because it doesn't seem to run on newer macs. So maybe, if this looks OK, you can merge it and I can iterate on remaining issues on a separate PR?

@fabianp
Copy link
Member

fabianp commented Mar 22, 2023

thanks Ariel!

@fabianp fabianp merged commit 0ec1a5a into openopt:master Mar 22, 2023
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.

2 participants