Skip to content

Automatic builds using Github Actions - #1071

Closed
Voker57 wants to merge 1 commit into
We-the-People-civ4col-mod:developfrom
Voker57:develop
Closed

Automatic builds using Github Actions#1071
Voker57 wants to merge 1 commit into
We-the-People-civ4col-mod:developfrom
Voker57:develop

Conversation

@Voker57

@Voker57 Voker57 commented Jan 24, 2025

Copy link
Copy Markdown
Contributor

This will build a mod archive from every commit to specified branch(es), and make a pre-release with file attached to it. You can see how it works here https://github.com/Voker57/wtp/releases/tag/development

Potential improvements:

  • A release is a >1GB file with most data unchanged. Nightingale suggested using some kind of incremental releases, this can be implemented if there's an agreement on how it should be done
  • Fastdep is not used because it doesn't work for some reason. It could theoretically speed up builds.

@Voker57
Voker57 force-pushed the develop branch 17 times, most recently from b0882b1 to 55a49b9 Compare January 30, 2025 13:46
@Voker57

Voker57 commented Jan 30, 2025

Copy link
Copy Markdown
Contributor Author

I have replaced the script hacks with use of environment variables to avoid asking for input. Incremental releases, after consideration, would require a rather complicated setup with questionable value, so this will be excluded from this PR. The request is ready to merge.

@Nightinggale

Copy link
Copy Markdown
Member

I'm not sure using compile.pl is the right move for this, but given that it reverts back to normal without specific environmental variables, I suppose it's doable. I will take a closer look at this when I get time to do so.

@Voker57
Voker57 force-pushed the develop branch 4 times, most recently from 00b204c to 681d70a Compare February 6, 2025 15:19
@Voker57
Voker57 force-pushed the develop branch 2 times, most recently from 67f5c57 to 540e092 Compare February 20, 2025 15:59
@Voker57
Voker57 force-pushed the develop branch 2 times, most recently from 94c2ff9 to d192a48 Compare March 2, 2025 10:29
@Voker57
Voker57 force-pushed the develop branch 3 times, most recently from 47780c6 to 41b5997 Compare March 10, 2025 18:03
@Voker57
Voker57 force-pushed the develop branch 3 times, most recently from fa4a45e to dfa3b03 Compare April 8, 2025 17:42
+ Create a pre-release from each commit, with downloadable mod build
* Modify build scripts not to ask for input if $NONINTERACTIVE is set
* Modify build scripts not to copy .dlls if $CI is set
@Voker57

Voker57 commented Apr 30, 2025

Copy link
Copy Markdown
Contributor Author

Superceded by #1122

@Voker57 Voker57 closed this Apr 30, 2025
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