rope icon indicating copy to clipboard operation
rope copied to clipboard

Migrate pytest.ini to pyproject.toml

Open gliptak opened this issue 3 years ago • 3 comments

Signed-off-by: Gábor Lipták [email protected]

Description

Please include a summary of the change and which issue is fixed.

Fixes #414

Checklist (delete if not relevant):

  • [x] I have added tests that prove my fix is effective or that my feature works
  • [x] I have updated CHANGELOG.md
  • [x] I have made corresponding changes to user documentation for new features
  • [x] I have made corresponding changes to library documentation for API changes

gliptak avatar Nov 26 '21 20:11 gliptak

Hi @gliptak, thanks for making this PR. The latest version of pytest that supports Python 2.7 unfortunately doesn't support putting configuration in pyproject.toml; so unfortunately, we'll have to keep the settings in pytest.ini until we drop Python 2.7 support officially.

lieryan avatar Nov 28 '21 11:11 lieryan

@lieryan would you be interested in that PR to drop already EOL 2.7 and 3.5 including soon to be EOL 3.6?

https://www.python.org/downloads/

gliptak avatar Nov 28 '21 11:11 gliptak

We want to drop 2.7 eventually, we have a rudimentary deprecation plan for Python 2.7 in https://github.com/python-rope/rope/issues/400. But for the moment we still maintain support for it. So let's keep this PR open until that moment comes

lieryan avatar Nov 29 '21 17:11 lieryan

Maybe update this to get the latest CI?

bagel897 avatar Nov 04 '22 20:11 bagel897

@bagel897 consider repushing as repo local branch

gliptak avatar Nov 04 '22 20:11 gliptak

I mean update this branch to the latest CI which is 3.7-3.11, just merge master. If not, I can make a new PR with the same stuff.

bagel897 avatar Nov 04 '22 21:11 bagel897

@bagel897 please make a new branch

gliptak avatar Nov 04 '22 21:11 gliptak

Moved to #519

bagel897 avatar Nov 04 '22 21:11 bagel897

Thanks for creating this PR @gliptak, thanks @bagel897 for updating this PR forwards.

lieryan avatar Nov 05 '22 14:11 lieryan