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

French translation #915

Merged
merged 79 commits into from
Mar 27, 2021
Merged

Conversation

forresst
Copy link
Contributor

This PR contains the complete translation of the README.md (README.french.md) and the complete French translation of 57 points (*.french.md). The remaining 29 points to be translated are a copy of the original English version in the *.french.md files).

This choice has been discussed with Yoni Goldberg.

It is possible to see the remaining translations here

Note that the files are up to date with the English version.

I want to thank @MrYamous and @tkhadir for their very precious help

* Translation of `dependencysecurity.md`

Start translation

* Finalizing the translation
* Translation of `secureheaders.md`

start of translations

* continue translations

* Finalizing the translation
Up to date on Feb 3, 2021
Up to date on Feb 14, 2021
* Translate readme (part6)

Start of translation

* Add translation 6.10 to 6.12

* Add translation 6.13 to 6.19

* Add translation 6.20 to 6.25
* Translate readme (part7)

Start of translation

* Fix link
* Translate readme (part8)

Start of translation

* Add translation 8.3 to 8.10

* Add translation 8.11 to 8.15
@goldbergyoni
Copy link
Owner

@forresst @tkhadir @MrYamous Formidable!💜

Amazing work. I used to know French, my family is from Marseille:), but I don't remember enough so @kevynb might have a quick look and approve

We should celebrate this translation by putting on news + tweeter after next week. Is there some Node.js group/form for Node.js to publish there?

Make sure to put your names on top of the French translation

@tkhadir
Copy link
Contributor

tkhadir commented Mar 15, 2021

thanks @goldbergyoni

All the merits go to @forresst 😄 , he is a committed person with whom we can carry out projects to end.

@forresst
Copy link
Contributor Author

We should celebrate this translation by putting on news + tweeter after next week. Is there some Node.js group/form for Node.js to publish there?

I guess you're looking for a French-speaking twetter account? Sorry I don't know any twetter.

Make sure to put your names on top of the French translation

I looked at the other translations to try to be consistent with the others. But I don't see exactly where to put it at the top?

Copy link
Owner

@goldbergyoni goldbergyoni left a comment

Choose a reason for hiding this comment

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

Looks super-awesome to me

@goldbergyoni goldbergyoni merged commit edf396e into goldbergyoni:master Mar 27, 2021
@goldbergyoni
Copy link
Owner

@all-contributors please add @tkhadir for content

@allcontributors
Copy link
Contributor

@goldbergyoni

I've put up a pull request to add @tkhadir! 🎉

@goldbergyoni
Copy link
Owner

@all-contributors please add @MrYamous for content

@allcontributors
Copy link
Contributor

@goldbergyoni

I've put up a pull request to add @MrYamous! 🎉

@goldbergyoni
Copy link
Owner

@all-contributors please add @MrYamous for content

@allcontributors
Copy link
Contributor

@goldbergyoni

@MrYamous already contributed before to content

@goldbergyoni
Copy link
Owner

@all-contributors please add @MrYamous for content

@allcontributors
Copy link
Contributor

@goldbergyoni

@MrYamous already contributed before to content

@goldbergyoni
Copy link
Owner

goldbergyoni commented Mar 27, 2021

@forresst

About your names at the top. -Just choose an adequate location

About forums - Is there any French newsgroup (e.g. Facebook/Discord), where we can promote this translation?

I'll take care to put in the news section soon

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

Successfully merging this pull request may close these issues.

4 participants