Skip to content

Conversation

@shifucun
Copy link
Contributor

No description provided.

@antaresc antaresc self-requested a review August 13, 2019 00:22
Copy link
Contributor

@antaresc antaresc left a comment

Choose a reason for hiding this comment

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

https://packaging.python.org/tutorials/packaging-projects/#setup-args

We can actually read in the LONG_DESCRIPTION from README.md like so in the example above.

@shifucun
Copy link
Contributor Author

Updated to use README.md for long description

@shifucun shifucun requested a review from antaresc August 13, 2019 18:10
setup.py Outdated
version=VERSION,
description=DESCRIPTION,
long_description=long_description,
long_description_content_type="text/markdown",
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: change double quotes to single quotes for consistency.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

Copy link
Contributor

@antaresc antaresc left a comment

Choose a reason for hiding this comment

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

PTAL at comment before merging. Thanks!

@shifucun shifucun merged commit cf2bd31 into datacommonsorg:master Aug 13, 2019
antaresc added a commit that referenced this pull request Oct 2, 2019
* Fix typo in places.py. (#71) (#84)

* Added iPynb notebooks and fixed issue #67 (#86)

* Added notebooks

* Added --upgrade flag

* Update requirements and setup.py

* Added maintenance information to the readme

* Update setup.py and pypi long description (#87)

* Update setup.py and requirements

* Add long description for pypi package

* Use README.md as the long description

* Use single quote

* Fixed docs and updated changelog (#89)

* Amended documentation for creating a key.

* Updated CHANGELOG

* Spacing

* Added 0.4.3 changes

* Added 0.x to changelog

* Add get_pop_obs api (#93)

* Add get_pop_obs api

* Updated comments with more detailed description.

* Update comment and rst file

* Updated comments

* Move to populations module

* Add missing change

* Correct typo

* More typo correction

* Add comments on observations field

* Add the step to join google group (#95)

* Add more details for joining google group (#96)

* Implemented get_place_obs and reimplemented query (#94)

* Implemented get_place_obs

* Reimplemented query as a function instead of a class.

* query now returns a list. Amended docstrings.

* Fixed docstring typo

* Add observation_date as argument to get_place_obs (#97)

* Add observation_date as argument to get_place_obs

* Update doc string

* Update test

* fix link to getting started on read the docs home page (#98)

* Fixed bug regarding indexing (#100)

* Updated the changelog (#101)
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