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

Remove latest version warning #302

Open
RobertCraigie opened this issue Sep 22, 2024 · 3 comments
Open

Remove latest version warning #302

RobertCraigie opened this issue Sep 22, 2024 · 3 comments
Labels
enhancement New feature or request

Comments

@RobertCraigie
Copy link
Owner

I suspect the latest version warning has caused more pain than usefulness, for a couple of reasons.

  • It's caused issues with flags like --outputjson which have since been fixed but it's a bit of a whack-a-mole situation and I wouldn't be surprised if other issues crop up in the future.
  • It makes the CLI slower, ~200ms on my machine which isn't too bad but it would be much slower if the user is on a slow network

Obviously with features like this one it's very hard, if not impossible, to actually judge how many people this has positively impacted. As such, I'm proposing removing the latest version warning log and if there is an uptick in pyright issues reported by users on an old version, we can consider adding it back in.

@RobertCraigie RobertCraigie added the enhancement New feature or request label Sep 22, 2024
@RobertCraigie
Copy link
Owner Author

cc @erictraut do you have any opinions here?

@erictraut
Copy link

I don't have a strong opinion here, but I worry a bit that removing this check will make it more likely that developers will have stale copies of pyright installed and not realize it.

One option to consider... you could keep the check in place but throttle it so it isn't performed on every invocation. For example, you could throttle it to once a week. That would require you to persist information about the time the last check was performed.

@ehiggs
Copy link

ehiggs commented Sep 23, 2024

Obviously with features like this one it's very hard, if not impossible, to actually judge how many people this has positively impacted. As such, I'm proposing removing the latest version warning log and if there is an uptick in pyright issues reported by users on an old version, we can consider adding it back in.

This is an important concern. While it's best to leave upgrades to the package manager (brew, etc), preventing spurious issues that drag down development it also important. However, this can be mitigated by adding an issue template that reminds people of the current version and that bugs filed against previous versions will not be treated with the same priority.

https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/configuring-issue-templates-for-your-repository

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants