Hello El 19/10/18 a las 12:19, Wouter Verhelst escribió: > On Thu, Oct 18, 2018 at 12:27:35PM +0200, Wouter Verhelst wrote: >> On Tue, Oct 16, 2018 at 11:52:55PM +0200, Laura Arjona Reina wrote: >>> * I'm not sure if we want that, or we prefer that the build fails and >>> make the CI not ignore these errors (then I guess the person committing >>> the file with wrong translation hash would get a mail, and hopefully fix >>> the file. If the page is built, maybe these kind of issues pass by >>> unnoticed). >> >> Yes. I think that should be done, indeed, and I've been working on it. > > FWIW, that's !33. I've been adding a "weak error checks" mode, where the > intention is that CI will fail, unless something is expected not to be > in the repository, such as the "Mirrors.masterlist" file, which seems to > be used all over the repository. >
Mirrors.masterlist file I think it's the easier one to get, it's in the repo https://salsa.debian.org/mirror-team/masterlist.git/ so we can get it cloning the repo and putting the file in /english/mirror folder. > One issue I'm running against now is /english/international/l10n/data; > that directory seems almost empty, but it would appear that it is > supposed to contain configuration data for the website itself (i.e., not > data that I would expect to find in another repository). > > Why is that not just added to git and committed? > The files are generated by cron scripts run daily: https://salsa.debian.org/webmaster-team/cron/blob/master/lessoften-parts/1l10n-data I have done some tests to include some parts of what is done via cron scripts to be done in .gitlab-ci.yml, to minimise the areas of webwml failing in the CI, but later I thought that if those processes are indeed in separated cron jobs that only run 6 times a day (and some others, once a day), maybe it's not reasonable to make them run in salsa with each commit... Kind regards -- Laura Arjona Reina https://wiki.debian.org/LauraArjona -- Laura Arjona Reina https://wiki.debian.org/LauraArjona Sent with K-9 mail