-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Description
Is your feature request related to a problem? Please describe.
#2267 introduced a new folder in the layout at the root of the repo, scripts
, to include a file that finds the most upvoted issues and sorts them in a Top-ranking issues tracker.
By default, our version manager, setuptools-scm
, includes all the repository files by default in the source distribution, sdist (that .tar.gz
file in PyPI). However, this source distribution often comprises just enough files to build a wheel, run tests, and create the documentation. Development-platform specific code and helpers are usually not considered in any of this categories; for example, .github
folder is pruned. Which folders/files get excluded is configured in MANIFEST.in
.
For example, using pvlib-0.13.0.tar.gz
as an example:
https://pypi.org/project/pvlib/#files

Describe the solution you'd like
Removing this folder as a whole, with the appropriate exclude instruction in MANIFEST.in
.
Describe alternatives you've considered
- Removing just the lone script file in that folder.
- Not doing anything, cause ya know, it doesn't really have an impact.
Additional context
I expect a contributor to deduce the exact instruction that must be used (it is easy from a quick glance at the manifest file).
Warning
I'm saving this issue for potential contributors that, (a) have never contributed to pvlib and (b) do expect to contribute to this library in the future. Please, don't work on this issue without previous agreement.