Skip to content

Conversation

@thewtex
Copy link
Member

@thewtex thewtex commented Apr 17, 2020

No description provided.

@thewtex
Copy link
Member Author

thewtex commented Apr 17, 2020

@hjmjohnson @mseng10 feel free to run with this.

Todos:

  • Address any new warnings
  • Add Windows build to the matrix
  • Add macOS build to the matrix
  • Add Action to build and generate Python package artifacts
  • Move to a dedicated GitHub Action Repository: this is not very feasible in a JavaScript Action.

The Windows and macOS configuration should be similar to Azure Pipelines.

@thewtex thewtex force-pushed the github-actions branch 13 times, most recently from fa0c17a to e6ec7e4 Compare April 18, 2020 14:20
@mseng10 mseng10 mentioned this pull request Apr 21, 2020
@thewtex
Copy link
Member Author

thewtex commented Apr 24, 2020

Failing Linux packaging issue is related to disk space issues: https://github.community/t5/GitHub-Actions/Failure-with-missing-logs/m-p/55055/highlight/false#M9401

@thewtex
Copy link
Member Author

thewtex commented Apr 24, 2020

disk space issues

In 5.1.0 we will remove the ITKPythonBuild tarball after it has been extracted to save disk space.

@mseng10
Copy link
Contributor

mseng10 commented Apr 24, 2020

@thewtex Just to clarify this would be ITKPythonBuilds-linux.tar.zst?

@thewtex
Copy link
Member Author

thewtex commented Apr 24, 2020

@mseng10 ITKPythonBuilds-linux.tar, which is ~5GB.

@thewtex thewtex force-pushed the github-actions branch 8 times, most recently from 80c4e96 to 9c29111 Compare April 30, 2020 01:39
@hjmjohnson
Copy link
Member

@thewtex Thank you for working on this. @mseng10 will follow your lead and implement this on other remote modules once it is working well.

@thewtex
Copy link
Member Author

thewtex commented Apr 30, 2020

@thewtex Thank you for working on this. @mseng10 will follow your lead and implement this on other remote modules once it is working well.

Awesome, thank you! That will be very helpful. CI for remote modules will not only working again after this is resolved, but it will be elegant and joyful to use 💖 .

@thewtex thewtex force-pushed the github-actions branch 16 times, most recently from 53c937f to a8aa0eb Compare May 2, 2020 00:37
@thewtex thewtex merged commit 1ae358d into master May 4, 2020
@thewtex thewtex deleted the github-actions branch May 4, 2020 02:46
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