Pre and post scripts order

[fakkio]

Hi,
what about move the pre- and post- scripts before and after the scripts they refer to respectively?

Fabio.

[notslang]

I think this is kinda a dupe of #33

[fakkio]

not quite,

my idea is to order script by alphabetical order but move pre and post script before and after the scripts they refer to, e.g.:

...,
"scripts": {
    "prebuild": "yarn fix-types",
    "build": "yarn build-index-css && react-scripts build",
    "postbuild": "node fix-index.js",
    "eject": "react-scripts eject",
    "fix-types": "node fix-types.js",
    "prestart": "yarn fix-types",
    "start": "react-scripts start",
    "test": "react-scripts test"
},
...

Here, the order is alphabetical (build, eject, fix-types, start, test) and the relative script are before and after them, it's not an user choice.

Fabio.

[pke]

Should that be configurable?

[jasonkarns]

Strongly agree with this. Having a dozen pre/post scripts moved away from the script they decorate is confusing and makes it hard to see cross-script dependencies. And because the name is forced on us by npm, we don't have any opportunity to control the order through naming.

@pke I don't think this kind of thing should be configurable. The majority of the value in tools like this is consistency. If each package needed to configure their own preferences then the consistency is lost. Should have reasonable defaults so no config is required.