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

PytestDeprecationWarning regarding DescribeBlock with pytest 5.4 #26

Closed
Cito opened this issue Mar 18, 2020 · 16 comments
Closed

PytestDeprecationWarning regarding DescribeBlock with pytest 5.4 #26

Cito opened this issue Mar 18, 2020 · 16 comments

Comments

@Cito
Copy link
Member

Cito commented Mar 18, 2020

With the latest pytest 5.4 I start getting the following warnings:

.tox/py36/lib/site-packages/pytest_describe/plugin.py:149: PytestDeprecationWarning:
  direct construction of DescribeBlock has been deprecated,
  please use DescribeBlock.from_parent return DescribeBlock(obj, collector)
@Cito
Copy link
Member Author

Cito commented Mar 18, 2020

Note that there are also warnings like PytestDeprecationWarning: TerminalReporter.writer attribute is deprecated, use TerminalReporter._tw, but I think this is a pytest issue.

@Cito
Copy link
Member Author

Cito commented Mar 18, 2020

See also the 5.4 list of deprecations regarding this change.

@ropez
Copy link
Collaborator

ropez commented Mar 27, 2020

@Cito Unfortunately, I haven't programmed I python in almost three years. If anyone is able to submit a PR, I will be happy to create a new version, but I'm afraid I won't have time to resolve these issues myself.

@Cito
Copy link
Member Author

Cito commented Mar 27, 2020

Too bad. But thank you for the feedback. Sadly, I cannot help either - too many other things going on already, and I fear this needs some deep diving into pytest plugin programming.

Please, if anybody else can help out here, that would be great.

@jacebrowning
Copy link
Contributor

I can take a look this weekend.

@jacebrowning
Copy link
Contributor

@ropez It's not a complete blocker, but getting #25 merged first will make fixing this warning easier.

@jacebrowning
Copy link
Contributor

jacebrowning commented Mar 28, 2020

#27 contains the patch.

I've confirmed this eliminates the PytestDeprecationWarning by installing the patch into a couple of my projects.

@ropez
Copy link
Collaborator

ropez commented Mar 30, 2020

@Cito @jacebrowning Would it be possible for any of you to just take over the responsibility for this? It has been just working for several years, so it's not really much to do, except for the occasional compatibility issue.

@Cito
Copy link
Member Author

Cito commented Mar 30, 2020

You mean the responsibility for the whole project? Including releasing on PyPI or would you still do that?

@ropez
Copy link
Collaborator

ropez commented Mar 31, 2020

Yes, I meant the whole project. I'm sure it's possible to add access to PyPi.

@Cito
Copy link
Member Author

Cito commented Mar 31, 2020

Yes, you can simply add new owners or maintainers (who can only publish but not delete releases) on PyPI. Regarding GitHub, it's probably best to transfer the project to the pytest-dev organization instead to another person. The current repository address ropez/pytest-describe will then be automatically forwarded by GitHub to pytest-dev/pytest-describe. I guess it's best to ask on [email protected] if they can initiate the transfer. Do you want to do that or shall I ask on your behalf? I can take over maintenance if nobody else volunteers, since I would hate to see this project die, but if @jacebrowning or somebody on the pytest-dev mailing list volunteers, or we could share the responsibility, I would be happier.

@jacebrowning
Copy link
Contributor

I'm not yet part of the pytest-dev organization, but I'm willing to help make releases if we can get it transferred there.

@ropez
Copy link
Collaborator

ropez commented Apr 2, 2020

@Cito Sounds great! I would be very happy if you could initiate this. I also wouldn't like to see this die, even if I'm not able to maintain it.

@Cito
Copy link
Member Author

Cito commented Apr 3, 2020

Ok, I just asked on the mailing list and hope we will hear back from someone in the pytest-dev org.

@Cito
Copy link
Member Author

Cito commented Apr 5, 2020

We now got a +1 from a pytest-dev developer, who also pointed to the official documentation for submitting plugins to pytest-dev, but we need another +1 to initiate the transfer.

You will also need to make @jacebrowning and me additional maintainers on PyPI, so that we have the minimum number of 3 people who can publish releases.

@Cito
Copy link
Member Author

Cito commented Apr 5, 2020

@ropez We now have two +1 from pytest-dev, so looks like we can start the transfer.

I have created a separate issue #30 to track the progress of the transfer, let's move this discussion there.

@Cito Cito closed this as completed in 453aa90 Apr 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants