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

sdist is missing test requirements file #7746

Closed
1 task done
mtelka opened this issue Oct 24, 2023 · 1 comment · Fixed by #7852
Closed
1 task done

sdist is missing test requirements file #7746

mtelka opened this issue Oct 24, 2023 · 1 comment · Fixed by #7852
Labels

Comments

@mtelka
Copy link
Contributor

mtelka commented Oct 24, 2023

Describe the bug

The sdist package at PyPI is missing requirements/test.txt and/or requirements/test.in file. Without them it is hard to know what Python projects are needed to successfully run test.

This causes some inconvenience during packaging of aiohttp for OpenIndiana since we are running tests to make sure our packages works properly.

To Reproduce

Download sdist from PyPI and unpack it.

Expected behavior

There is requirements/test.txt and/or requirements/test.in file.

Logs/tracebacks

N/A

Python Version

N/A

aiohttp Version

3.8.6

multidict Version

N/A

yarl Version

N/A

OS

OpenIndiana

Related component

Server, Client

Additional context

No response

Code of Conduct

  • I agree to follow the aio-libs Code of Conduct
@mtelka mtelka added the bug label Oct 24, 2023
@webknjaz
Copy link
Member

Fair. Integrating https://github.com/re-actors/checkout-python-sdist would be a welcome contribution if someone would want to make a PR.

Dreamsorcerer pushed a commit that referenced this issue Nov 19, 2023
patchback bot pushed a commit that referenced this issue Nov 19, 2023
Fixes #7746

(cherry picked from commit 122597f)
patchback bot pushed a commit that referenced this issue Nov 19, 2023
Fixes #7746

(cherry picked from commit 122597f)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants