Skip to content

Improvements to the workflow #245

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

Closed
wants to merge 151 commits into from

Conversation

derivativeoflog7
Copy link

I made a few changes I think can make compiling with GH Actions easier

  • Add the ability to upload the keystore as a base64 encoded string in the repo secrets, to avoid having to upload a keystore and having forks mess with branches or commits to make test builds; while fallbacking to using the keystore in the repo if the secret is not set
  • Defaulting USER_ID_OWNER to 0 if it's not set in the secrets (the feature is broken too as an int is too small to hold all possible user IDs, my own user ID breaks it) (why are there two features that have an override for the owner only anyway?)
  • Skip downloading the language file if the secret is not set
  • Add instructions

thermatk and others added 30 commits April 1, 2024 18:22
I guess upstream did the same and forgot to document.
isom.h is included in a form patched to stop annoying the compiler, dv.h in the original form
@23rd 23rd force-pushed the dev branch 2 times, most recently from 5c4539f to 6c19180 Compare December 7, 2024 00:03
@23rd 23rd force-pushed the dev branch 4 times, most recently from ea641d6 to 1aac211 Compare February 13, 2025 05:20
@23rd 23rd force-pushed the dev branch 3 times, most recently from 2df8bae to 4eda615 Compare March 12, 2025 10:58
@23rd 23rd force-pushed the dev branch 12 times, most recently from 8fb8a04 to 76239a6 Compare May 3, 2025 13:35
@23rd 23rd force-pushed the dev branch 5 times, most recently from 10cb3af to 5ced4d3 Compare June 5, 2025 00:25
@23rd 23rd closed this Jun 17, 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.

9 participants