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

Traduction installation.md #4

Merged
merged 3 commits into from
May 9, 2017
Merged

Conversation

MachinisteWeb
Copy link
Member

Revu du travail de traduction de @darenju

Comme d'habitude ont réaligne les termes avec la version principale où inversement et on vérifie que tout est à jour.

Signed-off-by: Bruno Lesieur <bruno.lesieur@gmail.com>
@MachinisteWeb MachinisteWeb self-assigned this May 8, 2017
@MachinisteWeb MachinisteWeb changed the title Traduction installation.md Traduction installation.md May 8, 2017
You will have to clone directly from GitHub and build `vuex` yourself if
you want to use the latest dev build.
Vous devrez cloner directement depuis GitHub et compiler `vuex` vous-même si
vous souhaitez utiliser la dernière version de dev.
Copy link
Member

Choose a reason for hiding this comment

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

Que penses-tu de

la dernière version en développement

?

Copy link
Member Author

@MachinisteWeb MachinisteWeb May 8, 2017

Choose a reason for hiding this comment

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

Il me semble que c'est ce qui a été utilisé pour vue-routeur ?

Sinon au mieux c'est build de développement dans le sens ou c'est un terme autorisé en temps que tel et déjà utilisé ultérieurement. (Un gros travail sur toutes les occurences de build et leur traduction sera surement à faire dans le futur. Éventuellement garder build nous permettra de rapidement le repérer dans la liste des « build » à traduire (compilation, version, package, construire, compiler, environnement, etc.).

Copy link
Member Author

Choose a reason for hiding this comment

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

Suite à la review de vue-router installation, je pense qu'aligner les deux de la sorte serait correcte

Vous aurez besoin de cloner directement vuex depuis GitHub et le compiler vous-même si vous souhaitez utiliser le dernier build de développement.

Copy link
Member Author

@MachinisteWeb MachinisteWeb left a comment

Choose a reason for hiding this comment

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

Parfait pour moi, rien à redire, juste du réalignement de termes pour la consistance cross documentation. Bien joué @darenju.

J'attends un peu plus d'avis avant de faire les modifs globals

@@ -36,12 +36,12 @@ import Vuex from 'vuex'
Vue.use(Vuex)
```

You don't need to do this when using global script tags.
Il n'est pas nécessaire de faire ceci lorsque vous utilisez des balises de script globales.
Copy link
Member Author

@MachinisteWeb MachinisteWeb May 8, 2017

Choose a reason for hiding this comment

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

Pour être au plus juste je propose de garder la personne you et aligner à vue-router

Vous n'avez pas besoin de faire cela lors de l'utilisation des balises de script globales (<script>).


### Dev Build
### Environnement de dev
Copy link
Member Author

@MachinisteWeb MachinisteWeb May 8, 2017

Choose a reason for hiding this comment

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

Nous ne concevrons généralement pas les raccourcis app => application, prod => production, etc. Donc

Et pour garder une consistance avec vue-router

Build de développement

(explication de pourquoi build) plus loin.

You will have to clone directly from GitHub and build `vuex` yourself if
you want to use the latest dev build.
Vous devrez cloner directement depuis GitHub et compiler `vuex` vous-même si
vous souhaitez utiliser la dernière version de dev.
Copy link
Member Author

@MachinisteWeb MachinisteWeb May 8, 2017

Choose a reason for hiding this comment

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

Il me semble que c'est ce qui a été utilisé pour vue-routeur ?

Sinon au mieux c'est build de développement dans le sens ou c'est un terme autorisé en temps que tel et déjà utilisé ultérieurement. (Un gros travail sur toutes les occurences de build et leur traduction sera surement à faire dans le futur. Éventuellement garder build nous permettra de rapidement le repérer dans la liste des « build » à traduire (compilation, version, package, construire, compiler, environnement, etc.).

Copy link
Member Author

@MachinisteWeb MachinisteWeb left a comment

Choose a reason for hiding this comment

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

J'ai fait une seconde passe après la review de l'installation de Vue Router avec des propositions pour que les originaux soient quasis identiques pour que les traductions concordent.


[https://unpkg.com/vuex](https://unpkg.com/vuex)

<!--email_off-->
[Unpkg.com](https://unpkg.com) provides NPM-based CDN links. The above link will always point to the latest release on NPM. You can also use a specific version/tag via URLs like `https://unpkg.com/vuex@2.0.0`.
[Unpkg.com](https://unpkg.com) fournit des liens CDN basés sur NPM. Le lien ci-dessus pointera toujours vers la dernière release sur NPM. Vous pouvez aussi utiliser un tag ou une version spécifique comme `https://unpkg.com/vuex@2.0.0`.
Copy link
Member Author

@MachinisteWeb MachinisteWeb May 8, 2017

Choose a reason for hiding this comment

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

Alignement avec vue-router (ajout de via une URL)

fournit des liens CDN basés sur NPM. Le lien ci-dessus pointera toujours vers la dernière release sur NPM. Vous pouvez aussi utiliser un tag ou une version spécifique via une URL comme

<!--/email_off-->

Include `vuex` after Vue and it will install itself automatically:
Incluez `vuex` après Vue et l'installation sera automatique :
Copy link
Member Author

Choose a reason for hiding this comment

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

Aligner avec vue-router

@@ -27,7 +27,7 @@ npm install vuex --save
yarn add vuex
```

When used with a module system, you must explicitly install Vuex via `Vue.use()`:
Lorsque vous utilisez un système de modules, vous devez explicitement installer le router via `Vue.use()`:
Copy link
Member Author

Choose a reason for hiding this comment

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

Conserver la troisième personne au début

Lorsqu'il est utilisé avec un système de module, vous devez explicitement installer Vuex via

Routeur à changer par Vuex

You will have to clone directly from GitHub and build `vuex` yourself if
you want to use the latest dev build.
Vous devrez cloner directement depuis GitHub et compiler `vuex` vous-même si
vous souhaitez utiliser la dernière version de dev.
Copy link
Member Author

Choose a reason for hiding this comment

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

Suite à la review de vue-router installation, je pense qu'aligner les deux de la sorte serait correcte

Vous aurez besoin de cloner directement vuex depuis GitHub et le compiler vous-même si vous souhaitez utiliser le dernier build de développement.

@MachinisteWeb MachinisteWeb removed the request for review from sylvainpolletvillard May 8, 2017 13:48
Signed-off-by: Bruno Lesieur <bruno.lesieur@gmail.com>
Signed-off-by: Bruno Lesieur <bruno.lesieur@gmail.com>
@MachinisteWeb
Copy link
Member Author

MachinisteWeb commented May 8, 2017

Prèt pour le merge, j'attends que tu me dises que ça a été ré-aligné avec l'installation de Vue-Router @Kocal !

@Kocal
Copy link
Member

Kocal commented May 8, 2017

@haeresis c'est bon, je viens de faire les modifs de mon côté (vuejs-fr/vue-router#3 (comment))

@MachinisteWeb MachinisteWeb merged commit 1c7d8fe into working May 9, 2017
@MachinisteWeb MachinisteWeb deleted the working_installation branch May 9, 2017 06:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants