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

bump pyright to 1.1.316 #19426

Merged
merged 2 commits into from Aug 3, 2023

Conversation

marijncv
Copy link
Contributor

@marijncv marijncv commented Jul 7, 2023

This PR bumps the default pyright version to 1.1.316 and takes into account the deprecation of the --venv-path parameter in favor of --venvpath (see here)

@huonw huonw requested a review from sureshjoshi July 14, 2023 03:40
@@ -152,21 +152,21 @@ def test_failing(rule_runner: PythonRuleRunner) -> None:
def test_multiple_targets(rule_runner: PythonRuleRunner) -> None:
rule_runner.write_files(
{
f"{PACKAGE}/good.py": GOOD_FILE,
f"{PACKAGE}/bad.py": BAD_FILE,
f"{PACKAGE}/bad1.py": BAD_FILE,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are tests covering "good" files, so that's alright that both of the files in this scenario are "bad".

Copy link
Member

@AlexTereshenkov AlexTereshenkov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@AlexTereshenkov AlexTereshenkov merged commit 156afe4 into pantsbuild:main Aug 3, 2023
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants