asttokens icon indicating copy to clipboard operation
asttokens copied to clipboard

Drop Python 2.7?

Open alexmojaki opened this issue 1 year ago • 6 comments

Continuing the discussion in https://github.com/gristlabs/asttokens/pull/110#issuecomment-1615147120

I've just noticed that GHA still has PyPy 2.7, which is actually probably enough to say that the library is still being tested on 2.7 and is thus expected to work. Dropping support (i.e. the PyPI classifier) is only valuable if it saves us from doing work to have that confidence.

On the other hand, I don't want people doing (or thinking that they have to do) extra work in the future for the sake of Python 2, e.g. I feel bad about the effort @palfrey went through with mypy and Python 2.

If we do go forward with this, https://github.com/gristlabs/asttokens/pull/67 would help.

alexmojaki avatar Jul 02 '23 11:07 alexmojaki

Cross linking: https://github.com/gristlabs/asttokens/issues/107

PeterJCLaw avatar Jul 02 '23 12:07 PeterJCLaw

@PeterJCLaw @dsagal

I just looked at Grist and their repo is setup with separate requirements.txt files for python 2 and python 3. As long as we don't change the public api, I think we will be fine with removing python 2 support.

Since the projects are supposedly tied, the way I suggest we go about it is:

  1. bump asttokens to version 3 and remove support for python 2
  2. release critical bug fixes to version 2.2.x if needed
  3. update requirements.txt to include asttokens <=2.0.5,<3
  4. update requirements3.txt to include the newer version 3

aqeelat avatar Jul 06 '23 08:07 aqeelat

Also, new grist installations use python 3.9 by default, so we can also drop 3.5 and 3.6 since they're dead.

aqeelat avatar Jul 06 '23 08:07 aqeelat

@aqeelat's suggested plan sounds good to me.

dsagal avatar Aug 29 '23 20:08 dsagal