Skip to content

Add long_description_content_type in setup.py#108

Merged
mdickinson merged 2 commits into
masterfrom
fix/long-description-content-type
Oct 10, 2019
Merged

Add long_description_content_type in setup.py#108
mdickinson merged 2 commits into
masterfrom
fix/long-description-content-type

Conversation

@rahulporuri
Copy link
Copy Markdown
Contributor

twine check was unhappy because of the missing long_description_content_type kwarg to setuptools.setup. This PR fixes that issue.

Weirdly enough, even after adding this kwarg, twine check is still unhappy because of a bug upstream in twine.

which was missing from setup.py

	modified:   setup.py
Copy link
Copy Markdown
Member

@mdickinson mdickinson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

twine check is working for me (twine version 2.0.0)

mirzakhani:apptools mdickinson$ twine check dist/apptools-4.5.0.dev63.tar.gz 
Checking dist/apptools-4.5.0.dev63.tar.gz: PASSED

@codecov-io
Copy link
Copy Markdown

codecov-io commented Oct 10, 2019

Codecov Report

Merging #108 into master will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #108      +/-   ##
==========================================
+ Coverage   41.67%   41.68%   +0.01%     
==========================================
  Files         240      240              
  Lines        8912     8912              
  Branches     1105     1105              
==========================================
+ Hits         3714     3715       +1     
  Misses       5044     5044              
+ Partials      154      153       -1
Impacted Files Coverage Δ
apptools/persistence/file_path.py 90.9% <0%> (+3.03%) ⬆️

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 99ef875...ea63cdb. Read the comment docs.

@mdickinson mdickinson merged commit 0b745d0 into master Oct 10, 2019
@mdickinson mdickinson deleted the fix/long-description-content-type branch October 10, 2019 14:58
mdickinson pushed a commit that referenced this pull request Oct 10, 2019
* FIX : Add long_description_content_type

which was missing from setup.py

	modified:   setup.py

* Add changelog entry.

(cherry picked from commit 0b745d0)
mdickinson pushed a commit that referenced this pull request Oct 10, 2019
* Release version 4.5.0 of apptools

	modified:   setup.py

* Add long_description_content_type in setup.py (#108)

* FIX : Add long_description_content_type

which was missing from setup.py

	modified:   setup.py

* Add changelog entry.

(cherry picked from commit 0b745d0)
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.

3 participants