Update --select arg to accept specific rules

[soulitzer]

No description provided.

[kit1980]

@soulitzer could you resolve the conflict so CI can run?

[soulitzer]

Updated!

[kit1980]

Now the tests are failing because https://github.com/pytorch-labs/torchfix/blob/main/tests/test_torchfix.py#L23 passes select="ALL", but now it's select: List[str].

Also select: List[str] = None will not pass a typechecker.

[soulitzer]

All the "ALL", default exclude, expansion handling is done in the arg processing/validation stage, so we don't need to deal with those later.

[kit1980]

Looks good overall, but there are couple of bugs:

$ torchfix . --select=ALL
Traceback (most recent call last):
  File "/home/sdym/.conda/envs/py39/bin/torchfix", line 8, in <module>
    sys.exit(main())
  File "/home/sdym/.conda/envs/py39/lib/python3.9/site-packages/torchfix/__main__.py", line 128, in main
    config.select = process_error_code_str(args.select)
  File "/home/sdym/.conda/envs/py39/lib/python3.9/site-packages/torchfix/__main__.py", line 34, in process_error_code_str
    raise ValueError(f"Invalid error code: {c}, available error "
ValueError: Invalid error code: ALL, available error codes: ['TOR003', 'TOR102', 'TOR002', 'TOR201', 'TOR202', 'TOR001', 'TOR101', 'TOR401']

$ torchfix . --select=TOR001
Traceback (most recent call last):
  File "/home/sdym/.conda/envs/py39/bin/torchfix", line 8, in <module>
    sys.exit(main())
  File "/home/sdym/.conda/envs/py39/lib/python3.9/site-packages/torchfix/__main__.py", line 128, in main
    config.select = process_error_code_str(args.select)
  File "/home/sdym/.conda/envs/py39/lib/python3.9/site-packages/torchfix/__main__.py", line 40, in process_error_code_str
    return list(expand_error_codes(raw_codes))
TypeError: unhashable type: 'list'

[soulitzer]

Damn maybe I should've ran the code wtf lol

[kit1980]

Thanks!