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

Add LANGUAGES=all to windows experimental release build #48763

Merged

Conversation

reasonablytall
Copy link
Contributor

Summary

None

Purpose of change

Based on feedback here, this updates the windows experimental release build to include all languages in the artifacts. This is an augment to #48705

Describe the solution

This adds the LANGUAGES=all flag to the build command.

Describe alternatives you've considered

Could leave the flag out, but we probably want all the languages to be included. This change brings this Action closer to the behavior of the jenkins build bot.

Testing

The workflow runs https://github.com/reasonablytall/Cataclysm-DDA/actions/runs/813316453

The artifacts are now the correct size (~60MB) and include the lang folder.

@actual-nh actual-nh added <Bugfix> This is a fix for a bug (or closes open issue) Code: Build Issues regarding different builds and build environments OS: Windows Issues related to Windows operating system Translation I18n labels May 5, 2021
@kevingranade kevingranade merged commit 5e8e372 into CleverRaven:master May 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bugfix> This is a fix for a bug (or closes open issue) Code: Build Issues regarding different builds and build environments OS: Windows Issues related to Windows operating system Translation I18n
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants