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

Enhancement: Add check for updates and allow for autoupdating inside spyder #2307

Closed
goanpeca opened this issue Apr 8, 2015 · 10 comments · Fixed by #2321
Closed

Enhancement: Add check for updates and allow for autoupdating inside spyder #2307

goanpeca opened this issue Apr 8, 2015 · 10 comments · Fixed by #2321
Assignees
Milestone

Comments

@goanpeca
Copy link
Member

goanpeca commented Apr 8, 2015

It would be very good to have a check for updates button under the help menu (i guess).

Also this check could be run on startup so the user gets a notification when an update is available and offer the install update option.

Now to handle the update we could:

  1. need a restart method for spyder (Enhancement: Add restart functionality to spyder. #2306),
  2. If using conda or anaconda we could run the StandAlone PackageManager with a flag to update spyder which would update spyder and on finish it would relaunch it and close itself.
  3. We could play with some task handling specific to each OS (seems to complicated..~!)

Comments?

@goanpeca goanpeca self-assigned this Apr 8, 2015
@goanpeca goanpeca added this to the v3.0 milestone Apr 8, 2015
@Nodd
Copy link
Contributor

Nodd commented Apr 8, 2015

Looks interesting. That would help decreasing bug reports from outdated versions of spyder.

@goanpeca
Copy link
Member Author

goanpeca commented Apr 8, 2015

Indeed! one of my major concerns with this

@ccordoba12
Copy link
Member

This is a good idea indeed! Perhaps the simplest thing to do to solve this one would be to use the Github API to check for the latest release and report that in a QMessageBox to our users.

The only thing is that I wouldn't like to rely on requests for that. So if you could use urllib or other standard lib module would be better.

@goanpeca
Copy link
Member Author

goanpeca commented Apr 8, 2015

Great :),

Just out of curiosity why not relying on requests?

@Nodd
Copy link
Contributor

Nodd commented Apr 8, 2015

To not add another dependency I guess. And marking it as optional would defeat the feature which is more oriented towards debutant users.

@ccordoba12
Copy link
Member

@Nodd seems to read my mind!! Yeap, that's exactly my thinking on requests for this.

@Nodd
Copy link
Contributor

Nodd commented Apr 8, 2015

import telepathy, the last version added quite a lot of new features !

@goanpeca
Copy link
Member Author

Ok, so on this one, the github api has some rate limits, right now it is 60... so if someone opens and closes spyder more than 60 times, the api would complain about it.

@ccordoba12
Copy link
Member

The rate limit restarts every hour. So for this to happen, users should open and close Spyder every minute ;-)

This also means that these checks shouldn't be applied when DEV == True.

@goanpeca
Copy link
Member Author

Implemented in #2321

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants