Skip to content

Use twine to check generated package and readme #1468

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

Merged
merged 1 commit into from
Apr 23, 2021

Conversation

francoisfreitag
Copy link
Contributor

The setup.py check command is deprecated. Use twine check instead.
Rename the tox environment to packaging, it now checks the package.

The setup.py check command is deprecated. Use twine check instead.
Rename the tox environment to packaging, it now checks the package.
@codecov
Copy link

codecov bot commented Apr 23, 2021

Codecov Report

Merging #1468 (29605f8) into main (7f2ad72) will decrease coverage by 0.16%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1468      +/-   ##
==========================================
- Coverage   86.47%   86.31%   -0.17%     
==========================================
  Files          35       35              
  Lines        1849     1849              
  Branches      258      258              
==========================================
- Hits         1599     1596       -3     
- Misses        178      181       +3     
  Partials       72       72              
Impacted Files Coverage Δ
debug_toolbar/panels/profiling.py 86.60% <0.00%> (-2.68%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7f2ad72...29605f8. Read the comment docs.

@matthiask matthiask merged commit 33c5e53 into django-commons:main Apr 23, 2021
@matthiask
Copy link
Member

Thanks!

@francoisfreitag francoisfreitag deleted the twine-check branch April 23, 2021 11:46
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.

2 participants