-
-
Notifications
You must be signed in to change notification settings - Fork 475
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
makefile #2
Comments
Thanks. @gedw99 |
https://github.com/gedw99/sc-make/blob/main/assets/.sc-make/make/go.mk#L113 Shows one way to switch based on OS. You can then modify the BIN names based on the OS . Like needing .exe for windows |
I don't seem to need this makefile, I can implement it via Action to compile all versions automatically every time I push. |
All the the GitHub Workflows can just call your make targets too. example: https://github.com/gedw99/sc-make/blob/main/.github/workflows/ci.yaml The CI is calling this: https://github.com/gedw99/sc-make/blob/main/makefile#L17 |
It’s better imho to do the opposite. the GitHub action I gave you is generic and calls the makefike. the advantage is that whatever working on your laptop is exactly what happens on GitHub CI |
Can you create a new pull request? I can learn it. |
you can put this into repo if you want.
everything works.... your description in the readme make it easy to make this work.
MAC only. can make it work on Windows and Linux with a bit more work.
The text was updated successfully, but these errors were encountered: