Skip to content

Update update-tolgee.yml: Run every Monday#3247

Closed
Adam Stachowicz (Saibamen) wants to merge 1 commit intoDevolutions:mainfrom
Saibamen:patch-1
Closed

Update update-tolgee.yml: Run every Monday#3247
Adam Stachowicz (Saibamen) wants to merge 1 commit intoDevolutions:mainfrom
Saibamen:patch-1

Conversation

@Saibamen
Copy link
Contributor

Reduce git pull requests and commits

Reduce git pull requests and commits
@marticliment
Copy link
Collaborator

Why?

@Saibamen
Copy link
Contributor Author

Adam Stachowicz (Saibamen) commented Feb 3, 2025

To reduce pull requests you need to merge and to reduce commits in changelog view/latest commits view.

Right now there are 261 commits between 3.1.5 and 3.1.6-beta3: 3.1.5...3.1.6-beta3

And 30 of them are for translations. Lot of them are only updating one or two languages.

@marticliment
Copy link
Collaborator

Martí Climent (marticliment) commented Feb 3, 2025

And 30 of them are for translations. Lot of them are only updating one or two languages.

Yes, but updated translations take effect on already published versions of UniGetUI. Therefore, by decreasing the amount of PRs the changes would take much longer to be effective.

Once a PR is merged, UniGetUI will pull new translations from GitHub

@Saibamen
Copy link
Contributor Author

Adam Stachowicz (Saibamen) commented Feb 3, 2025

Once a PR is merged, UniGetUI will pull new translations from GitHub

Ahh ok. Translations are not included in UnigetUI source code, but downloaded from GH and cached on local disk.

@marticliment
Copy link
Collaborator

Martí Climent (marticliment) commented Feb 4, 2025

Yes, exactly (although a backup is included on source code in case the device fails to download translations)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants