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.

Sign up for GitHub

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

Jump to bottom

Fix readthedocs build #110

Merged
merged 21 commits into from
Apr 8, 2020
Merged

Fix readthedocs build #110

merged 21 commits into from
Apr 8, 2020

Conversation

mcflugen
Copy link
Member

@mcflugen mcflugen commented Apr 7, 2020

This pull request was originally intended to fix broken builds on readthedocs, which it does, but while I was at it, I figured I would also clean up the CI and fix an issue with the pymt.cfunits package.

@mcflugen
add requirements files for development and testing
a3af3e2
@mcflugen
configure readthedocs build
e036fc1
@mcflugen
include requirements file and license in MANIFEST
e7122af
@mcflugen
update requirements files
2087203
@mcflugen
read install requirements from requirements file
1d4e1f5
@mcflugen
update CI to use pip; build with python 3.8
82e5690
@mcflugen
matplotlib-base is not on pypi
9e0cda2
@mcflugen
require a beta version of landlab
264fa4d
@mcflugen
run conda init bash
23b330e
@mcflugen
require landlab 2, including pre releases
0b92128
@mcflugen
source .bashrc after conda init
fa6c2cf
@mcflugen
install esmpy from conda-forge
33b2184
@mcflugen
install udunits2 from conda-forge
7cab70c
@mcflugen
remove pip --pre option
7eca330
@mcflugen
source .bash_profile on mac
8194ee1
@mcflugen
fix typo
d53443d
@mcflugen
point to pymt.cfunits xml data files
7d0231a
@mcflugen
use miniconda on appveyor
293aa1b
@mcflugen
install testing requirements on appveyor
e80c87a
@mcflugen
install all requirements using conda
f47edef
@mcflugen
install geos, udunits2 with conda, everything else with pip
429f4f8
@mcflugen mcflugen requested a review from mdpiper April 7, 2020 21:44
@mcflugen mcflugen self-assigned this Apr 7, 2020
mdpiper
mdpiper approved these changes Apr 7, 2020
View reviewed changes
Copy link
Member

@mdpiper mdpiper left a comment

Choose a reason for hiding this comment

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

@mcflugen This is neat because it provides an advanced example of building (both code & docs), testing, and deploying a Python package. I'll try to use it!

@mcflugen mcflugen merged commit 880a6cc into master Apr 8, 2020
@mcflugen mcflugen deleted the mcflugen/fix-rtfd-build branch April 8, 2020 03:56
@mcflugen mcflugen mentioned this pull request Apr 8, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mdpiper mdpiper mdpiper approved these changes

Assignees

@mcflugen mcflugen

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mcflugen @mdpiper