Skip to content
This repository has been archived by the owner on Dec 22, 2021. It is now read-only.

Switching over to setuptools #294

Merged
merged 2 commits into from Jul 14, 2016
Merged

Switching over to setuptools #294

merged 2 commits into from Jul 14, 2016

Conversation

leouieda
Copy link
Member

@leouieda leouieda commented Jul 13, 2016

The Python Packaging Guide recommends using setuptools instead of distutils in setup.py.

This is a one line replacement and adds the advantage of the find_packages function (use it instead of having to list all subpackages in Fatiando by hand).

Checklist:

  • Make tests for new code (at least 80% coverage)
  • Create/update docstrings
  • Include relevant equations and citations in docstrings
  • Docstrings follow the style conventions
  • Code follows PEP8 style conventions
  • Code and docs have been spellchecked
  • Include new dependencies in doc/install.rst, requirements.txt, environment.yml, ci/requirements-conda.txt and ci/requirements-pip.txt.
  • Documentation builds properly (run cd doc; make locally)
  • Changelog entry (leave for last to avoid conflicts)

@leouieda leouieda merged commit 03810c3 into master Jul 14, 2016
@leouieda leouieda deleted the use-setuptools branch July 14, 2016 14:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant