Skip to content

docs: consolidate dependency management into top-level config#111

Merged
ITProKyle merged 1 commit intomasterfrom
feature/consolidate-venvs
Aug 19, 2021
Merged

docs: consolidate dependency management into top-level config#111
ITProKyle merged 1 commit intomasterfrom
feature/consolidate-venvs

Conversation

@ITProKyle
Copy link
Copy Markdown
Owner

Summary

Remove the need to manage two different virtual environments (app & docs). The dependencies to build docs are now managed from the top-level pyproject.toml file.

These dependencies are added as extras due to a limitation of ReadTheDocs (does not support poetry) and poetry (can't install dev dependencies using pip).

What Changed

Added

  • added docs extra dependencies to top-level pyproject.toml

Changed

  • poetry will now remove untracked dependencies when running make setup
  • poetry will install the docs extra dependencies when running make setup

Removed

  • removed dedicated venv/dependency management used for docs

@ITProKyle ITProKyle self-assigned this Aug 19, 2021
@github-actions github-actions Bot added feature Request or pull request for a new feature poetry Update to Poetry configuration python Update to Python code labels Aug 19, 2021
@ITProKyle ITProKyle merged commit 35d4c04 into master Aug 19, 2021
@ITProKyle ITProKyle deleted the feature/consolidate-venvs branch August 19, 2021 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Request or pull request for a new feature poetry Update to Poetry configuration python Update to Python code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant