Skip to content

feat: tooling upgrades, strict docstrings, and --version flag - #61

Merged
JacksonFergusonDev merged 4 commits into
mainfrom
misc
Mar 4, 2026
Merged

feat: tooling upgrades, strict docstrings, and --version flag#61
JacksonFergusonDev merged 4 commits into
mainfrom
misc

Conversation

@JacksonFergusonDev

Copy link
Copy Markdown
Owner

Description

This PR updates project tooling configurations, enforces stricter documentation standards, and introduces a version output flag to the CLI.

Changes

  • Tooling: Bumped pre-commit hooks (ruff and markdownlint-cli) to their latest upstream versions.
  • Typing & Linting: Updated pyproject.toml to enforce Google-style docstrings (ruff pydocstyle plugin) and configured mypy to emit more verbose context. Fixed trailing D205 docstring errors.
  • CLI: Added a --version flag to the main parser. This dynamically resolves the package version using importlib.metadata, eliminating the need for hardcoded version strings.
  • Config: Corrected a namespace typo in .github/ISSUE_TEMPLATE/config.yml pointing to jacksonfergusondev.

Validation

  • Run uv run ruff check . (0 errors)
  • Run mypy
  • Test git pulsar --version outputs correct metadata

- Update ruff-pre-commit to v0.15.4
- Update markdownlint-cli to v0.48.0
Updates contact_links URL to point to the correct jacksonfergusondev namespace.
- Update pyproject.toml to include ruff 'D' (pydocstyle) rules with Google convention.
- Resolve resulting D205 formatting errors in cli.py and git_wrapper.py.
- Enable verbose output and error context for mypy.
Implements standard argparse versioning using importlib.metadata to dynamically pull the package version from the installed wheel/sdist.
@JacksonFergusonDev
JacksonFergusonDev merged commit ea1a656 into main Mar 4, 2026
4 checks passed
@JacksonFergusonDev
JacksonFergusonDev deleted the misc branch March 4, 2026 00:56
@codecov

codecov Bot commented Mar 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

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