[jira] [Commented] (CB-12810) keep existing formatting in package.json

["Duy Dao (JIRA)" <ji...@apache.org>]

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

Duy Dao commented on CB-12810:
------------------------------

@Doug Davies The autosave / config is not respected in the prepare operation. AFAIK the only way to get rid of it atm, is to keep the plugins in sync with the generated file that will be generated by cordova prepare, so it won't get marked for changes.

> keep existing formatting in package.json
> ----------------------------------------
>
>                 Key: CB-12810
>                 URL: https://issues.apache.org/jira/browse/CB-12810
>             Project: Apache Cordova
>          Issue Type: Improvement
>          Components: cordova-lib
>    Affects Versions: 7.0.0
>            Reporter: Steve Gill
>            Priority: Minor
>              Labels: backlog, cordova-8.0.0, cordova-next
>
> We will write our own indentation in package.json. For commands that shell out to cordova-fetch and therefore npm, we have no control over it (cordova platform/plugin add/rm).
> We could theoretically keep indentaiton for commands where we write to package.json (also platform/plugin add/rm to cordova key & during restore when we sync config.xml & package.json).
> This issue is probably not worth implementing. 
> npm@5 seems like it will respect existing formatting. 
> Previous issue with npm formatting: https://github.com/npm/npm/issues/5694
> npm5 respecting indentation: https://github.com/npm/npm/issues/16244



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

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