Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pyright job is flaky and temporarily optional #1121

Closed
hynek opened this issue Apr 5, 2023 · 3 comments
Closed

Pyright job is flaky and temporarily optional #1121

hynek opened this issue Apr 5, 2023 · 3 comments
Labels
Meta Issues regarding building, packaging, shipping, ...

Comments

@hynek
Copy link
Member

hynek commented Apr 5, 2023

Presumably due to some hosting changes, nodeenv and by extension Pyright are regularly failing now.

I've marked it as optional for now, but would like to re-activate it ASAP.

Ref ekalinin/nodeenv#324

@hynek hynek added the Meta Issues regarding building, packaging, shipping, ... label Apr 5, 2023
@webknjaz
Copy link
Member

webknjaz commented Apr 5, 2023

Instead of commenting out the job dependency, you can use an action input to mark when it's okay for a job to fail FYI

@hynek
Copy link
Member Author

hynek commented Apr 5, 2023

If you wanna open a PR, feel free. For me that was easier than reading docs. ..🤪

@hynek
Copy link
Member Author

hynek commented May 22, 2023

Looks like pyright is now really breaking as of presumably 1.1.309:

E         Full diff:
E           {
E         +  PyrightDiagnostic(severity='error', message='Cannot assign member "a" for type "Frozen"\n\xa0\xa0"Frozen" is frozen'),
E         +  PyrightDiagnostic(severity='information', message='Type of "DefineConverter.__init__" is "(self: DefineConverter, with_converter: int) -> None"'),
E         +  PyrightDiagnostic(severity='information', message='Type of "Define.__init__" is "(self: Define, a: str, b: int) -> None"'),
E         +  PyrightDiagnostic(severity='error', message='Cannot assign member "a" for type "FrozenDefine"\n\xa0\xa0"FrozenDefine" is frozen'),
E         +  PyrightDiagnostic(severity='information', message='Type of "af.__init__" is "(_a: int) -> None"'),
E            PyrightDiagnostic(severity='information', message='Type of "d2.a" is "Literal[\'new\']"'),
E         -  PyrightDiagnostic(severity='information', message='Type of "af.__init__" is "(_a: int) -> None"'),
E         -  PyrightDiagnostic(severity='error', message='Cannot assign member "a" for type "FrozenDefine"\n\xa0\xa0"FrozenDefine" is frozen'),
E         -  PyrightDiagnostic(severity='error', message='Cannot assign member "a" for type "Frozen"\n\xa0\xa0"Frozen" is frozen'),
E         -  PyrightDiagnostic(severity='information', message='Type of "Define.__init__" is "(self: Define, a: str, b: int) -> None"'),
E         -  PyrightDiagnostic(severity='information', message='Type of "DefineConverter.__init__" is "(self: DefineConverter, with_converter: str | bytes | bytearray | memoryview | array[Any] | mmap | _CData | PickleBuffer | SupportsInt | SupportsIndex | SupportsTrunc) -> None"'),
E            PyrightDiagnostic(severity='information', message='Type of "d.a" is "Literal[\'new\']"'),
E           }

tests/test_pyright.py:85: AssertionError

Volunteers? 😇🤓

@hynek hynek closed this as completed Nov 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Meta Issues regarding building, packaging, shipping, ...
Projects
None yet
Development

No branches or pull requests

2 participants