You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cordova.apache.org by "Kerri Shotts (JIRA)" <ji...@apache.org> on 2017/05/08 13:41:04 UTC

[jira] [Commented] (CB-12766) Cordova CLI changes package.json indentation

    [ https://issues.apache.org/jira/browse/CB-12766?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16000742#comment-16000742 ] 

Kerri Shotts commented on CB-12766:
-----------------------------------

A quick search (https://github.com/apache/cordova-lib/search?utf8=✓&q=fs.writeFileSync+stringify&type=) reveals that there are several places where package.json is written out with four spaces instead of two. NPM itself doesn't respect spacing and always writes out two spaces, so we may as well follow.

> Cordova CLI changes package.json indentation
> --------------------------------------------
>
>                 Key: CB-12766
>                 URL: https://issues.apache.org/jira/browse/CB-12766
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: cordova-cli, cordova-lib
>    Affects Versions: 7.0.0
>            Reporter: Remco Haszing
>              Labels: easy-fix
>
> While Cordova is busy adding platforms / plugins, it writes to package.json several times. In this process the indentation of package.json alternates between 2 and 4 spaces. The final result is an indentation of 4 spaces, which conflicts with the indentation of 2 spaces that npm uses.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org