Skip to content

Comments

Work/load translations early#11300

Merged
TheOneRing merged 3 commits intomasterfrom
work/load-translations-early
Oct 27, 2023
Merged

Work/load translations early#11300
TheOneRing merged 3 commits intomasterfrom
work/load-translations-early

Conversation

@erikjv
Copy link
Contributor

@erikjv erikjv commented Oct 18, 2023 •
edited by fmoc
Loading

Remove the --confdir option and load translations before any messages are shown.

Fixes #11246. Fixes #11142.

Waits on #11316.

@erikjv erikjv requested review from TheOneRing and fmoc October 18, 2023 13:44
@erikjv erikjv self-assigned this Oct 18, 2023
@erikjv
Copy link
Contributor Author

erikjv commented Oct 18, 2023

This still needs changelog entries

@erikjv erikjv force-pushed the work/load-translations-early branch from 90cd708 to 42a5a00 Compare October 18, 2023 14:28
@erikjv
Copy link
Contributor Author

erikjv commented Oct 18, 2023

This still needs changelog entries

Done.

TheOneRing
TheOneRing reviewed Oct 26, 2023
View reviewed changes
@TheOneRing
Copy link
Contributor

TheOneRing commented Oct 26, 2023

@saw-jan I just realised the gui tests are relying on --confdir could you elaborate why?

@saw-jan
Copy link
Member

saw-jan commented Oct 26, 2023

@saw-jan I just realised the gui tests are relying on --confdir could you elaborate why?

yeah, we are using that option to use different config dir other than default one so that we don't mess up configs while running tests locally.

Fabian suggested if we can use XDG_CONFIG_HOME env during tests. so, we are going to try that one

@TheOneRing
Copy link
Contributor

TheOneRing commented Oct 26, 2023

@saw-jan I just realised the gui tests are relying on --confdir could you elaborate why?

yeah, we are using that option to use different config dir other than default one so that we don't mess up configs while running tests locally.

Fabian suggested if we can use XDG_CONFIG_HOME env during tests. so, we are going to try that one

thx can you prepare a pr? (please don't commit to this one)

@saw-jan
Copy link
Member

saw-jan commented Oct 26, 2023 •
edited
Loading

thx can you prepare a pr? (please don't commit to this one)

yeah, on the way

@saw-jan saw-jan mentioned this pull request Oct 26, 2023
Merged
@saw-jan
Copy link
Member

saw-jan commented Oct 26, 2023

yeah, on the way

PR is ready to be merged: #11316

erikjv added 3 commits October 27, 2023 15:07
@erikjv
Remove deprecated option --confdir
7f99f88 
This was meant to be used before roaming profiles were supported on
Windows.

Fixes: #11246
@erikjv
Do translation setup before command-line option parsing
552c0e3 
Now the messages during early start-up can be localized.

Fixes: #11142
@erikjv
Make command-line options and messages translatable
8b4766c
@erikjv erikjv force-pushed the work/load-translations-early branch from 40b1243 to 8b4766c Compare October 27, 2023 13:08
@erikjv erikjv requested a review from TheOneRing October 27, 2023 13:09
@TheOneRing TheOneRing enabled auto-merge (rebase) October 27, 2023 13:57
@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 27, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 14 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@TheOneRing TheOneRing merged commit bbb63d8 into master Oct 27, 2023
@delete-merged-branch delete-merged-branch bot deleted the work/load-translations-early branch October 27, 2023 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fmoc fmoc Awaiting requested review from fmoc

1 more reviewer

@TheOneRing TheOneRing TheOneRing approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@erikjv erikjv

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Deprecate legacy --confdir command-line argument Setup translations before Application instantiation

3 participants

@erikjv @TheOneRing @saw-jan