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

Add py to requirements #227

Merged
merged 1 commit into from
Oct 25, 2022
Merged

Add py to requirements #227

merged 1 commit into from
Oct 25, 2022

Conversation

abey79
Copy link
Contributor

@abey79 abey79 commented Oct 25, 2022

This is needed since py is used but not explicitly listed. Pytest 7.2 no longer requires py, which breaks pytest-benchmark with this version of pytest.

Fixes #226

This is needed since `py` is used but not explicitly listed. Pytest 7.2 no longer requires `py`, which breaks pytest-benchmark with this version of pytest.

Fixes #226
@abey79
Copy link
Contributor Author

abey79 commented Oct 25, 2022

Note: I used the same version as in pytest 7.1.x.

@abey79
Copy link
Contributor Author

abey79 commented Oct 25, 2022

Also note this security advisory regarding py (including the latest version):
https://nvd.nist.gov/vuln/detail/CVE-2022-42969

@ionelmc ionelmc merged commit 073d7ed into ionelmc:master Oct 25, 2022
@abey79 abey79 deleted the patch-1 branch October 25, 2022 09:17
@ahopkins
Copy link

Thanks for the quick work. Any timeline for release?

@ionelmc
Copy link
Owner

ionelmc commented Oct 25, 2022

I'll get on it this week, gotta freshen up that CI :-)

gadomski added a commit to stac-utils/pystac-client that referenced this pull request Nov 15, 2022
Required to picked up ionelmc/pytest-benchmark#227 to get pytest 7.2.0 working.
Thanks to pytest-dev/pytest#10420 for pointing the right way.
gadomski added a commit to stac-utils/pystac-client that referenced this pull request Nov 15, 2022
)

* build(deps-dev): update pytest requirement from ~=7.1.3 to ~=7.2.0

Updates the requirements on [pytest](https://github.com/pytest-dev/pytest) to permit the latest version.
- [Release notes](https://github.com/pytest-dev/pytest/releases)
- [Changelog](https://github.com/pytest-dev/pytest/blob/main/CHANGELOG.rst)
- [Commits](pytest-dev/pytest@7.1.3...7.2.0)

---
updated-dependencies:
- dependency-name: pytest
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>

* deps, dev: bump pytest-benchmark to 4.0.0

Required to picked up ionelmc/pytest-benchmark#227 to get pytest 7.2.0 working.
Thanks to pytest-dev/pytest#10420 for pointing the right way.

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Pete Gadomski <pete.gadomski@gmail.com>
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 this pull request may close these issues.

Compatibility issue with pytest 7.2: Package py is used but not listed as dependency
3 participants