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

gfmpw1 run-precheck Makefile target uses -it, breaks github actions #341

Open
mattvenn opened this issue Nov 15, 2023 · 2 comments
Open
Assignees

Comments

@mattvenn
Copy link
Contributor

Since commit 9ee2462 by @marwaneltoukhy

-it has been added to the docker precheck target.

What is the reasoning behind that? It means the precheck now fails in a github action.

@marwaneltoukhy
Copy link
Member

It was added so that docker runs interactively so you can kill the process if needed, when running locally.

@mattvenn
Copy link
Contributor Author

interesting. That doesn't actually work for me, I have to open a new terminal and docker kill it. Is there a way where it could tell if it has terminal and only then uses -it? For now I'm just removing -it so I can run it as a GH action: https://github.com/mattvenn/project0_test/actions/runs/6878330117/workflow#L43

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

No branches or pull requests

2 participants