Remove old perl script to update l10n files#6265
Conversation
|
@MorrisJobke, thanks for your PR! By analyzing the history of the files in this pull request, we identified @DeepDiver1975, @jakobsack and @jancborchardt to be potential reviewers. |
6def65a to
81b21f6
Compare
Codecov Report
@@ Coverage Diff @@
## master #6265 +/- ##
============================================
- Coverage 51.69% 51.69% -0.01%
Complexity 25382 25382
============================================
Files 1599 1599
Lines 95090 95090
Branches 1376 1376
============================================
- Hits 49161 49160 -1
- Misses 45929 45930 +1
|
|
Can we avoid the new folder in root? |
The idea is to drop the l10n because in all other cases (all apps out there and also all folders in the server) this holds the actual l10n stuff - except in the root of the server, which is quite cumbersome. Also the logic of the extraction code now uses this folder as indication of the root and therefore is a lot easier to understand, because this is the only way it works and it works for all apps and the server. I will do additional tests later on this. |
|
Okay then ping @LukasReschke to adjust the packaging script |
Once it is ready :) |
81b21f6 to
b00771d
Compare
|
CI not happy |
I know - release script needs to be touched as well. Let me do a final test |
Signed-off-by: Morris Jobke <[email protected]>
b00771d to
23f9ef5
Compare
|
@rullzer @nickvergessen This is now good to go in. Please review |
Thanks @jakobsack