8000 Disable all transforms by default · Issue #134 · lebab/lebab · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
Disable all transforms by default #134
Closed
@nene

Description

@nene

I've been actually pondering over this topic for a while...

I think it's not really safe to enable all the transforms by default, and we should discourage users from doing it. I've already restructured the README to point out three transforms that aren't fully safe to perform, and even with others my suggestion is to only perform one transform at the time.

I think we should bake this into Lebab executable itself, so it would require listing the explicit transforms you'd like to make. So that if you just run:

$ lebab myfile.js

You'd get an error:

No transforms specified. Specify at least one of the transforms from the list below:
- ...docs for all the transforms...

I would not even add an explicit option for enabling all transforms, as I'd like Lebab to encourage performing one transform at a time.

This will also pave a way for optional transforms like #133 which not everybody might want to have.

It will also remove the need for separate --enable and --disable options. As there won't be any default transforms to disable, we can have just one option like --transform aka -t (which would be imho better mnemonic than --enable).

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      0