luttik / auto-optional Goto Github PK

Add an exclude option

Consider:

• Using glob-patterns
• Looking at other formatters and linters for the best patterns

Leave a 👍 if you want this

Add an in-line ignore option.

Support:

• # noqa
• # auto-optional: ignore

Leave a 👍 if you want this

Union[None, str] should not become Optional[Union[None, str]]

This does not make sense.

auto-optional==0.1.11
python 3.9.7

Hi @Luttik ,

I am trying to use it in an existing project and I am adding next config to my .pre-commit-config.yaml file.
The idea that I want to specify path to command so it checks src/ folder only
Here is an overview structure of project:

...
├── requirements-dev.in
├── requirements-dev.txt
├── requirements.in
├── requirements.txt
├── src
├── static
├── templates
├── tmp
...

This is config for auto-optional which I integrate into .pre-commit-config.yaml file

          ......
        - id: auto-optional
          name: auto-optional
          description: Adds the Optional type-hint to arguments where the default value is None
          entry: auto-optional src/
          language: python
          types: [python]
          ....

And I am getting next errors:

Error: Got unexpected extra arguments (src/asgi.py src/accounts/models/bill_blacklist_models.py src/hooks/models.py .... (continue list of files which should be proceeded)

How do I resolve it:

Added pass_filenames: false so in the end the config should be:

        - id: auto-optional
          name: auto-optional
          description: Adds the Optional type-hint to arguments where the default value is None
          entry: auto-optional src/
          language: python
          types: [python]
          pass_filenames: false

It takes longer time on the whole for verifying but this is a temp solution for that.

Support other file-formats than UTF-8 for python code

Leave a 👍 if you want this

Add a diff, dry-run or check command for linting.

Preferably:

• --check

Leave a 👍 if you want this.