mquinson / po4a-website

Source code of https://po4a.org/
GNU General Public License v2.0
5 stars 5 forks source link

Updated Italian translation #5

Closed ciampix closed 6 years ago

ciampix commented 6 years ago

Updated Italian translation

bexelbie commented 6 years ago

@mquinson ditto here re: guide mentioned in weblate update PR (other repo)

ciampix commented 6 years ago

sorry I do not understand you @bexelbie ... what do you mean?

bexelbie commented 6 years ago

@ciampix sorry was pointing @mquinson at my comment on https://github.com/mquinson/po4a/pull/79

We should have a maintainers checklist to ensure that everyone who has commit rights manages translation updates the exact same way.

ciampix commented 6 years ago

Just for the site translation purpose I can migrate to weblate if this simplify the translation workflow to all... btw I have no commit rights on the web pages of po4a ... just on the po4a source repo (a bit weird...)

mquinson commented 6 years ago

Hello,

@ciampix I just granted you the commit right on both repo.

@bexelbie I usually push all changes as soon as I get something. I agree that it's a good idea to add a mention about it in the README file. Please be my guest :) I'll do it later this week if you dont beat me on this.

ciampix commented 6 years ago

@mquinson many thanks! Re-reading my post found it a bit obscure. What I was trying to say (in a way a bit clumsy I may say...) is that my translation was accepted asking for a pull so it is possible to commit without having commit rights at all; that is probably another possible work-flow and it could also be safe to do it in this way if we manage to put in use a CI system to check for consistency of the po files before merging them... never did it myself but I know for sure that github supports it.

bexelbie commented 6 years ago

Following up on what @ciampix said, it would be ideal to have a PR based workflow if we can do CI testing on things like po files and then potentially auto-merge or bot-merge them.

I know the CI system we are looking at will support this idea, but it won't be ready immediately.