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

new-feature: create binaries with GitHub Actions #142

Merged
merged 64 commits into from
Sep 25, 2024

Conversation

uriii3
Copy link
Collaborator

@uriii3 uriii3 commented Sep 18, 2024

Add GitHub Actions so that the binaries are created at every release or pre-release.

  • MacOS binary
  • Linux binary
  • Windows binary
  • Make sure the imports are correct
  • Make sure the --add-data option works
  • See where it saves the binaries
  • Maybe also add tests to make sure the binaries work properly

@uriii3 uriii3 changed the base branch from main to copernicusmarine-toolbox-v2 September 18, 2024 14:20
@uriii3 uriii3 requested a review from renaudjester September 20, 2024 08:47
Comment on lines +108 to +114
python -m pip install --upgrade pip
pip install pyinstaller
pip install -e .
pip install poetry
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can take this part and put it into a make command

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I still think that it would be better to do all of this into a mamba env

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also with the mamba env it seems like you have access to a power-shell for windows

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is already into a makefile, no? I don't understand what you mean.

Yes, but I was a bit scared because I don't fully understand the mamba things so I'm not so sure that it would work better... seems to work fine like this!

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah I know the mamba thing can be some work but it would be nice because it keeps environment installation the same across the repo
In the repo, for github action we never use pure pip commands

@uriii3 uriii3 force-pushed the creating_executable branch from bb37a68 to 79ab6db Compare September 25, 2024 08:04
@uriii3 uriii3 merged commit 839a9a3 into copernicusmarine-toolbox-v2 Sep 25, 2024
2 checks passed
@uriii3 uriii3 deleted the creating_executable branch September 25, 2024 12:59
renaudjester added a commit that referenced this pull request Oct 28, 2024
renaudjester added a commit that referenced this pull request Oct 28, 2024
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