Skip to content

Standardize development dependency location + use uv more#1734

Open
bmos wants to merge 19 commits intomove-coop:mainfrom
bmos:pep-735
Open

Standardize development dependency location + use uv more#1734
bmos wants to merge 19 commits intomove-coop:mainfrom
bmos:pep-735

Conversation

@bmos
Copy link
Contributor

@bmos bmos commented Feb 20, 2026

Moved requirements-dev.txt dev dependencies into the build-system.requires and dependency-groups sections of pyproject.toml, as recommended in Python Packaging User Guide > Dependency Groups documentation.

Benefits:

  • Follow current python packaging standards for build and development dependencies
  • Reduce runtime of each pytest CI instance by not installing unused dev dependencies
  • Allow installing parsons and development dependencies with a single command
  • Parallels Friendly dependencies as default behavior #1721, which eliminates requirements.txt, but for dev dependencies

@bmos bmos marked this pull request as draft February 20, 2026 22:45
@bmos bmos marked this pull request as ready for review February 20, 2026 23:33
bmos and others added 2 commits February 24, 2026 20:35
* Fix missing string closure in write_tests.rst

* Fix sphinx warning "title overline too short"

* autoclass :: -> autoclass::

* Fix sphinx warning "title underline too short"

* Fix sphinx autodoc AttributeError parsing utilities.dbt

* Fix sphinx error "Error in "code-block" directive"

* Fix sphinx critical error unexpected section title from improper use of markdown code blocks in reStructuredText

* Fix sphinx error "Unexpected indentation"

* Fix sphinx warning "Inline strong start-string without end-string"

* Fix sphinx warning "Duplicate explicit target name"

* add missing toc entries

* remove trailing whitespace from all files

* ensure all files have newline at end of file

* convert tabs to spaces in remaining two files

* fix improperly-formatted links in zoom documentation

* remove unecessary parentheses in method reference

* replace tab characters with spaces in getting_set_up.rst

* `Args:` -> `Args`: -> Args: (and Returns)

* fix no-blank-line-after-section

* fix no-blank-line-before-section

* fix missing-blank-line-after-last-section

* fix overindented-section

* install censusgeocode

* fix note syntax

* fix autoclass references

* resolve attempted slackclient import

* add mock import for pytz and airtable

* fix MobilizeAmerica path

* fix inherited-members indentation

* don't use autodoc module/class name hinting

* add :members: to all autoclasses

* try to fix remaining autodoc import warnings

* save docs-build artifact on circleci (for testing)

* resolve SyntaxWarning for \* in tofrom.py

* fix artifact upload path

* improve docstring formatting of actblue.py

* resolving more docstring syntax errors

* Inline strong start-string without end-string.

* Duplicate explicit target name

* Unknown target name

* ERROR: Unknown directive type "notes".

* Block quote ends without a blank line

* fix Unexpected indentation

* Duplicate explicit target name

* Enumerated list ends without a blank line

* Title underline too short

* autodoc: failed to import class 'dbtRunnerParsons

* Error parsing content block for the "list-table" directive: two-level bullet list expected, but row 4 does not contain a second-level bullet list.

* Definition list ends without a blank line

* unexpected unindent.

* Bullet list ends without a blank line

* link formatting

* fix Unknown target name: "user"

* Inline strong start-string without end-string

* fixing a lot of errors and warnings

* fix malformed table

* exclude sphinx output from git

* Upgrade CircleCI config

* more docstring formatting

* improve return/yield docstring formatting

* don't try to overwrite existing .venv in circleci

---------

Co-authored-by: Shauna Gordon-McKeon <shaunagm@gmail.com>
@bmos bmos force-pushed the pep-735 branch 2 times, most recently from 31b96e2 to f79e326 Compare March 12, 2026 03:52
@github-actions
Copy link

github-actions bot commented Mar 12, 2026

Coverage report

This PR does not seem to contain any modification to coverable code.

@bmos bmos force-pushed the pep-735 branch 4 times, most recently from 346da31 to d7f8175 Compare March 12, 2026 21:13
@bmos bmos force-pushed the pep-735 branch 2 times, most recently from 97a1313 to 074f07d Compare March 13, 2026 01:53
bmos and others added 4 commits March 12, 2026 22:35
* remove circleci documentation workflow

* move makefile logic to make.py and update sphinx config

* create new documentation workflow

* add sphinx-lint dependency, job, and pre-commit hook

* add final newline, trailing whitespace, and yaml+toml syntax checks to pre-commit workflow

* resolve unsupported theme option 'display_version' given

* update source_suffix from list to dictionary

* add sphinx workflow cache

* exclude sphinx outputs from git

* fix copying of index redirect

* indentation for index.html

* use furo theme

* add furo to requirements-dev

* keep old build files < 5.0.0 between runs

* Update dependency-review.yml to whitelist url and remove GHSA allowance

Removed specific GHSA allowance from dependency review action.

* add pydocstyle ruff rules

* reduce_rows raw docstring

* run pre-commit

* Delete .circleci directory

* add make.py and conf.py to sphinx cache key

---------

Co-authored-by: Shauna Gordon-McKeon <shaunagm@gmail.com>
@bmos
Copy link
Contributor Author

bmos commented Mar 13, 2026

@shaunagm merge conflicts resolved!

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.

1 participant