Skip to content

Conversation

@lucaspar
Copy link
Member

  • New pre-commit action that runs deptry on the gateway side to track missing and unused dependencies.
    • Added rule exceptions to packages that Django calls dynamically and are not explicitly imported.
  • New pre-commit action to prevent large files in git history.
  • Updated django doc URLs to open the docs of the current v4.2 in use.
  • Removed deprecated requirements/* files.

@lucaspar lucaspar self-assigned this Jan 26, 2026
Copilot AI review requested due to automatic review settings January 26, 2026 16:23
@lucaspar lucaspar added ci/cd GH actions, tests, builds, packaging, etc gateway Gateway component dependencies Pull requests that update a dependency file labels Jan 26, 2026
@semanticdiff-com
Copy link

semanticdiff-com bot commented Jan 26, 2026

Review changes with  SemanticDiff

Changed Files
File Status
  gateway/config/settings/base.py  34% smaller
  gateway/package-lock.json  26% smaller
  gateway/config/settings/production.py  1% smaller
  .pre-commit-config.yaml  0% smaller
  gateway/.envs/example/django.prod-example.env Unsupported file format
  gateway/config/asgi.py  0% smaller
  gateway/config/settings/local.py  0% smaller
  gateway/config/settings/test.py  0% smaller
  gateway/docs/changelog.md Unsupported file format
  gateway/justfile Unsupported file format
  gateway/pyproject.toml Unsupported file format
  gateway/requirements/base.txt Unsupported file format
  gateway/requirements/local.txt Unsupported file format
  gateway/requirements/production.txt Unsupported file format
  gateway/sds_gateway/users/models.py  0% smaller
  gateway/uv.lock Unsupported file format
  sdk/src/spectrumx/errors.py  0% smaller

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR modernizes the Python dependency management for the gateway project by introducing automated dependency tracking with deptry, updating documentation URLs to match the Django 4.2 version in use, and removing deprecated requirements files in favor of the uv-based workflow.

Changes:

  • Adds deptry pre-commit hook and configuration to track missing and unused dependencies
  • Updates all Django documentation URLs from /en/dev/ to /en/4.2/ to match the project's Django version
  • Removes deprecated gateway/requirements/*.txt files (base.txt, local.txt, production.txt)

Reviewed changes

Copilot reviewed 9 out of 11 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
sdk/src/spectrumx/errors.py Improves type hint for Unset singleton by using Self instead of string literal
gateway/uv.lock Updates multiple dependencies including boto3, coverage, django-allauth, gunicorn, and others
gateway/requirements/* Removes deprecated requirements files now superseded by pyproject.toml
gateway/pyproject.toml Adds deptry configuration with DEP002/DEP003 ignore rules, updates fido2 version, adds requests dependency
gateway/config/settings/base.py Updates Django documentation URLs to v4.2 and adds clarifying comments
.pre-commit-config.yaml Adds check-added-large-files hook and deptry pre-commit hook
gateway/justfile Adds deptry command for manual dependency checking
gateway/docs/changelog.md Reformats text to improve line wrapping
gateway/package-lock.json Updates npm dependencies including parcel/watcher, memfs, and others
Files not reviewed (1)
  • gateway/package-lock.json: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@lucaspar lucaspar requested a review from klpoland January 26, 2026 16:43
@lucaspar lucaspar merged commit 3618e72 into master Jan 27, 2026
5 checks passed
@lucaspar lucaspar deleted the lp/gwy-deps branch January 27, 2026 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/cd GH actions, tests, builds, packaging, etc dependencies Pull requests that update a dependency file gateway Gateway component

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants