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

[cookbook/debugging-in-vs-code] Translated in french #203

Merged
merged 2 commits into from
Dec 16, 2019
Merged

[cookbook/debugging-in-vs-code] Translated in french #203

merged 2 commits into from
Dec 16, 2019

Conversation

SamuelAlev
Copy link
Member

No description provided.

@Vinetos
Copy link
Member

Vinetos commented Dec 3, 2019

#166 si tu veux une autre version

@MachinisteWeb
Copy link
Member

Navré pour le travail en double @Vinetos, si tu veux jeter un œil en tant que reviewer avant qu'on merge la PR, n'hésite pas.

@SamuelAlev
Copy link
Member Author

@haeresis @Vinetos
Le truc c'est que ta PR change cookbook/using-axios-to-consume-apis.md et non cookbook/debugging-in-vs-code 😅

Copy link

@forresst forresst left a comment

Choose a reason for hiding this comment

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

Merci pour ton travail, j'ai plusieurs remarques qui me semblent bon à modifier

src/v2/cookbook/debugging-in-vscode.md Outdated Show resolved Hide resolved
src/v2/cookbook/debugging-in-vscode.md Outdated Show resolved Hide resolved

Before you can debug your Vue components from VS Code, you need to update the generated Webpack config to build sourcemaps. We do this so that our debugger has a way to map the code within a compressed file back to its position in the original file. This ensures that you can debug an application even after your assets have been optimized by Webpack.
Avant de pouvoir déboguer vos composants Vue à partir de VS Code, vous devez mettre à jour la configuration webpack générée pour construire des sourcemaps. Nous faisons cela pour que notre débogueur ait un moyen de mapper le code d'un fichier compressé à sa position dans le fichier d'origine. Ceci vous permet de déboguer une application même après que vos ressources aient été optimisées par webpack.
Copy link

Choose a reason for hiding this comment

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

Suggested change
Avant de pouvoir déboguer vos composants Vue à partir de VS Code, vous devez mettre à jour la configuration webpack générée pour construire des sourcemaps. Nous faisons cela pour que notre débogueur ait un moyen de mapper le code d'un fichier compressé à sa position dans le fichier d'origine. Ceci vous permet de déboguer une application même après que vos ressources aient été optimisées par webpack.
Avant de pouvoir déboguer vos composants Vue à partir de VS Code, vous devez mettre à jour la configuration Webpack générée pour construire des sourcemaps. Nous faisons cela pour que notre débogueur ait un moyen de mapper le code d'un fichier compressé à sa position dans le fichier d'origine. Ceci vous permet de déboguer une application même après que vos ressources aient été optimisées par Webpack.

Copy link
Member Author

Choose a reason for hiding this comment

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

Salut, justement à cause de #4 j'ai retiré la majuscule.
Je suis d'accord avec toi pour la majuscule 🙂

Copy link

Choose a reason for hiding this comment

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

En fait, après quelques recherches, je m'aperçois qu'on doit plutôt écrire "webpack" sans la majuscule. Je pense qu'il faudrait le laisser en minuscule et demander une correction sur l'original.
@haeresis ton avis sur ce point ?

Copy link
Member

Choose a reason for hiding this comment

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

Oui webpack divrait rester sans majuscule tant que la documentation officielle n'en utilisera pas :)

Mon avis sur la question : https://www.linkedin.com/pulse/d%C3%A9veloppeurs-marketeurs-respectez-les-noms-des-bruno-lesieur/

Bien entendu cette avis ne reste pas absolue et on ne suit pas uniquement mon avis ici. Donc si on a de bonnes raisons de faire autrement ou de mauvaise raison de faire autrement (pression de groupe basé sur des préférences personnelles partagés par un grand nombre) on pourra toujours changer ça :)

src/v2/cookbook/debugging-in-vscode.md Outdated Show resolved Hide resolved
src/v2/cookbook/debugging-in-vscode.md Outdated Show resolved Hide resolved
src/v2/cookbook/debugging-in-vscode.md Outdated Show resolved Hide resolved
src/v2/cookbook/debugging-in-vscode.md Outdated Show resolved Hide resolved
src/v2/cookbook/debugging-in-vscode.md Outdated Show resolved Hide resolved
src/v2/cookbook/debugging-in-vscode.md Outdated Show resolved Hide resolved
src/v2/cookbook/debugging-in-vscode.md Outdated Show resolved Hide resolved
@MachinisteWeb MachinisteWeb merged commit 9030bf1 into vuejs-fr:master Dec 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants