Skip to content
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

PyPi packaging and cleanup. #3

Merged
merged 1 commit into from
Nov 7, 2011
Merged

PyPi packaging and cleanup. #3

merged 1 commit into from
Nov 7, 2011

Conversation

gtaylor
Copy link
Contributor

@gtaylor gtaylor commented Nov 7, 2011

  • Converted README.md to ReST, since that's what PyPi uses for their
    detail pages.
  • Created a top-level wepay module, containing api and exceptions.
    This is to facilitate the packaging process.
  • ReSTified all docstrings so Sphinx docs can be generated.
  • Added a setup.py file. DEFAULTED TO BSD LICENSE. MAKE SURE TO CHANGE THIS TO WHATEVER YOU WANT.
  • Added a VERSION const to wepay.init.py. PyPi wants this.
    Feel free to change the value.
  • Moved WePayError into its own module.
  • Eliminated a few unsafe default kwarg values in WePay class.
    Dicts can be accidentally manipulated if left as default.

* Converted README.md to ReST, since that's what PyPi uses for their
  detail pages.
* Created a top-level wepay module, containing api and exceptions.
  This is to facilitate the packaging process.
* ReSTified all docstrings so Sphinx docs can be generated.
* Added a setup.py file.
* Added a VERSION const to wepay.__init__.py. PyPi wants this.
  Feel free to change the value.
* Moved WePayError into its own module.
* Eliminated a few unsafe default kwarg values in WePay class.
  Dicts can be accidentally manipulated if left as default.
@gtaylor gtaylor mentioned this pull request Nov 7, 2011
@maxcountryman
Copy link

Excellent! Looks much improved.

brycec pushed a commit that referenced this pull request Nov 7, 2011
PyPi packaging and cleanup. Thanks gtaylor! license to come.
@brycec brycec merged commit c14c3b4 into wepay:master Nov 7, 2011
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.

None yet

3 participants