Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

2019 Fonts chapter translation to French #2114

Merged
merged 20 commits into from
May 4, 2021
Merged

Conversation

arigaud-ca
Copy link
Contributor

@arigaud-ca arigaud-ca commented Apr 3, 2021

First PR for Fonts 2019
I will go through another round of proofreading

Makes progress on #539

@tunetheweb
Copy link
Member

It looks like this translation was started some time ago, and there have been many changes to the chapter since then - primarily in the meta data and the figure format. I think I still have the regexs to converts the figures so if you're OK with me doing that I can commit to this branch to bring the formats up to date with the latest version?

@tunetheweb tunetheweb added the translation world wide web label Apr 4, 2021
@tunetheweb tunetheweb added this to the 2020 Content Translation milestone Apr 4, 2021
@tunetheweb tunetheweb requested a review from a team April 4, 2021 10:39
@arigaud-ca
Copy link
Contributor Author

@bazzadp sure you can commit to this branch or do you prefer that I start again forking a more recent branch from the project?

@tunetheweb
Copy link
Member

@bazzadp sure you can commit to this branch or do you prefer that I start again forking a more recent branch from the project?

Brought it up to date there.

Also staged a version here so you can see what it looks like: https://20210404t165715-dot-webalmanac.uk.r.appspot.com/fr/2019/fonts

Comment on lines +328 to +336
"arigaud-ca": {
"name": "Arthur Rigaud",
"teams": [
"translators"
],
"avatar_url": "https://avatars2.githubusercontent.com/u/50932885?v=4&s=200",
"github": "arigaud-ca",
"twitter": "arigaud_ca"
},
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@arigaud-ca can you confirm you're happy with this info? and or change it as you see fit.

@nico3333fr
Copy link
Contributor

Some tiny typos and harmonisations mentionned... you did great job @arigaud-ca ! :)

@tunetheweb
Copy link
Member

Thanks for reviewing @nico3333fr !

@arigaud-ca , there's only a few small things to clean up here and then I think this is good to merge. Will you have a chance to close this out so we can release this?

@tunetheweb tunetheweb changed the title Fonts 2019 Fonts chapter translation to French Apr 22, 2021
@tunetheweb
Copy link
Member

@arigaud-ca will you have time to finish this off? Very close now so would be great to be able to launch it.

@tunetheweb
Copy link
Member

I've addressed most of the feedback so there really only are a few small things to fix here.

@nico3333fr would you be able to finish this out if possible since @arigaud-ca seems unavailable? Keen to get this merged so it doesn't become stale or hold up any other changes.

I've staged what the chapter currently looks like here: https://20210502t102616-dot-webalmanac.uk.r.appspot.com/fr/2019/fonts

If there's no further update in next few days then I'll merge it as is and we can iterate on it in the future.

@nico3333fr
Copy link
Contributor

@bazzadp I can try between tonight and tomorrow. ;)

@tunetheweb
Copy link
Member

Thanks mate. Appreciated. Will hold off merging.

@nico3333fr
Copy link
Contributor

@bazzadp added 2 suggestions.

@tunetheweb tunetheweb merged commit 69a82b8 into HTTPArchive:main May 4, 2021
@nico3333fr
Copy link
Contributor

@bazzadp when I'll have the time, I'll re-read it again to have the "full picture" and do the typo-nazi, but better release, this is already a really good work :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
translation world wide web
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants