Skip to content

Conversation

@sc0tts
Copy link

@sc0tts sc0tts commented Feb 18, 2019

As it is, the environment.yml file in the pymt/docs/ folder does not contain package "pymt" as a dependency. However, that is required in order to "make html".

This pull request simply adds "pymt" to the list of packages in the docs/environment.yml file.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.07%) to 71.171% when pulling d4340d4 on sc0tts:sc0tts/update_doc_env_yml into a8cbfe7 on csdms:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.07%) to 71.171% when pulling d4340d4 on sc0tts:sc0tts/update_doc_env_yml into a8cbfe7 on csdms:master.

@mdpiper
Copy link
Member

mdpiper commented Feb 18, 2019

Pymt is installed through setuptools at the repo level.

@mcflugen
Copy link
Member

@mdpiper is correct. We don't want to add pymt to docs/environment.yml. When we build the docs we want to build them based on the pymt source code, not the latest release.

@mcflugen mcflugen closed this Feb 18, 2019
mdpiper added a commit to sc0tts/pymt that referenced this pull request Feb 19, 2019
This reverts commit d4340d4.
See the resolution to csdms#66.
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.

4 participants