DEV: Include CHANGELOG.md and LICENSE in tool.flit.sdist

[j-t-1]

No description provided.

[stefan6419846]

Could you please elaborate on this as requested in one of your previous PRs? Especially since it already works correctly:

[j-t-1]

Can ".flake8", ".pylintrc" and "tox.ini" be removed from the exclude array of tool.flit.sdist?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.48%. Comparing base (67e1781) to head (69fc476).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3101   +/-   ##
=======================================
  Coverage   96.48%   96.48%           
=======================================
  Files          52       52           
  Lines        8811     8811           
  Branches     1614     1614           
=======================================
  Hits         8501     8501           
  Misses        185      185           
  Partials      125      125           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[j-t-1]

Could you please elaborate on this as requested in one of your previous PRs? Especially since it already works correctly:

This is trying to remove the need for MANIFEST.in. I do not fully understand the build process so this PR may need further scrutiny.

[stefan6419846]

I recommend extending the existing CI workflows to check for some of the expected files in both distributions in the package step and remove the MANIFEST file if everything is working correctly.

[j-t-1]

I recommend extending the existing CI workflows to check for some of the expected files in both distributions [Windows and non-Windows?] in the package step and remove the MANIFEST file if everything is working correctly.

I did a test removing the contents in MANIFEST.in, but not amending anything else, and it passes the existing tests.

[stefan6419846]

This should be an automated test, not a manual one. And with distributions, I mean the sdist and bdist.

[j-t-1]

This should be an automated test, not a manual one. And with distributions, I mean the sdist and bdist.

May need some help with this, as still not understanding the process. Is this correct:

bdist: .github/workflows/publish-to-pypi.yaml
sdist: pyproject.toml

[stefan6419846]

The workflow file you are mentioning and the pyproject.toml file should be unrelated to this, unless I am misunderstanding something.

When running python3 -m build, you should get a source (sdist, tar file) and binary (bdist, wheel file) distribution. The github-ci.yaml workflow should check that the correct files are available in the corresponding generated files, id est the sdist should contain both files, while the bdist should not contain the changelog.