Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

Update French translation #871

Closed
wants to merge 1 commit into from
Closed

Update French translation #871

wants to merge 1 commit into from

Conversation

yannicka
Copy link
Contributor

@yannicka yannicka commented Oct 8, 2018

No description provided.

@christianbundy
Copy link
Contributor

christianbundy commented Oct 8, 2018

I'm not sure how many changes were made, but it looks like it's a full-file diff because the indentation was changed from spaces to tabs. I think it might be best to replace the new tabs with their original indentation. Looking forward to seeing the new translations, thanks!

@mmckegg
Copy link
Contributor

mmckegg commented Oct 9, 2018

I see that there were also some missing i18n from the recent tags PR and the refactor going on right now. I'd recommend in the future that do changes that are not directly part of the language update as a seperate pull request just to keep things nice a clean. And as @christianbundy mentioned, it looks like your editor may have converted the indentation to tabs.

But I'm going to merge this anyway (and clean it up manually)!

@mmckegg mmckegg closed this in 96e51fc Oct 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants