-
Notifications
You must be signed in to change notification settings - Fork 62
[Tracker] refactor npm scripts #2376
Comments
To further expand the command primitives with some initial set of modifier parameters:
UPDATE: revisit |
@debloper how is this different from what we already have? |
@nimishamukherjee as discussed, this:
In addition tho those, as good-to-have, it also
Also, about the tests - you are correct at this point in time. But once we're done with standalone-planner, we need to accommodate for Functional tests as it's not only a library anymore, but a full user facing app. It's given, that some of the parameters mentioned there are far into the future (coverage, code quality, security tests etc), but by design this system already makes space for it when it needs to be implemented. |
While the npm-scripts that we have in Planner repo is pretty lean, and is limited bunch of commands - it never hurts to have a preset plan to make them more manageable and cope with further addition of capabilities without bloating it up.
Right now, the scripts' block looks like this:
The necessary and sufficient primary tasks (primitives, not specialized) that we need, would be:
Let's spare describing them as they should be pretty self explanatory.
In order to work with specialized tasks (clean:all or test:unit and the likes of them), how about we use
minimist
oryargs
withgulp
(which we already use), and pass the specialization factors as parameters to the main script (whichnpm
allows with--
).This will allow us to abstract away the specific commands from
package.json
as well as where we use them, leaving only the task handlers (and in some cases parameters). The specifics of how the task is run and which tools are used/replaced will have no impact on them, as long as they can take the command+param and execute it.The text was updated successfully, but these errors were encountered: