Skip to content

feat: separate dependency specification#96

Merged
szymonmaszke merged 6 commits intomainfrom
94
Feb 26, 2026
Merged

feat: separate dependency specification#96
szymonmaszke merged 6 commits intomainfrom
94

Conversation

@szymonmaszke
Copy link
Member

Checklist

  • I agree to follow this project's Code of Conduct
  • I have read this project's Contributing Guide
  • I have created relevant issue(s) and linked them in the PR description

Closes #94

Copilot AI review requested due to automatic review settings February 26, 2026 19:55
@github-actions github-actions bot added security Security updates deps Dependencies updates config Configuration files updates feat Feature labels Feb 26, 2026
Copy link

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 pull request separates dependency specification from pyproject.toml into a dedicated dependencies/dependencies.txt file, switching the build backend from pdm-backend to hatchling with the hatch-requirements-txt plugin. This change aims to simplify dependency management and reduce the complexity of the pyproject.toml file.

Changes:

  • Switched build backend from pdm-backend to hatchling with hatch-requirements-txt plugin
  • Created dependencies/dependencies.txt for dependency specification (currently empty)
  • Updated documentation to reference the new dependency location
  • Updated dependency versions in pdm.lock
  • Added package overrides in osv-scanner.toml for new dev dependencies
  • Changed line length from 80 to 110 characters

Reviewed changes

Copilot reviewed 5 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
pyproject.toml Changed build backend to hatchling, configured dynamic dependencies, removed inline dependency list, updated line length to 110
dependencies/dependencies.txt New empty file for project dependencies
pdm.lock Updated dependency versions across the board
osv-scanner.toml Added license ignore overrides for griffelib, peewee, and python-discovery
SECURITY-DEPENDENCY.md Updated documentation to mention both pyproject.toml and dependencies.txt
README.md Updated usage instructions to reference dependencies.txt instead of pyproject.toml
.yamllint.yml Increased max line length from 80 to 110
Comments suppressed due to low confidence (1)

pyproject.toml:288

  • The line length is changed from 80 to 110 characters, which is also reflected in .yamllint.yml. However, this change is not mentioned in the PR description or linked issue #94. This appears to be an unrelated change that should either be explained or separated into its own PR.
line-length = 110

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

Signed-off-by: szymonmaszke <github@maszke.co>
@github-actions github-actions bot added the actions GitHub Actions updates label Feb 26, 2026
Signed-off-by: szymonmaszke <github@maszke.co>
Signed-off-by: szymonmaszke <github@maszke.co>
@github-actions github-actions bot added tests Tests updates docs Documentation updates labels Feb 26, 2026
Signed-off-by: szymonmaszke <github@maszke.co>
@szymonmaszke szymonmaszke merged commit 6389155 into main Feb 26, 2026
39 of 72 checks passed
@szymonmaszke szymonmaszke deleted the 94 branch February 26, 2026 20:38
Signed-off-by: szymonmaszke <github@maszke.co>
Signed-off-by: szymonmaszke <github@maszke.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

actions GitHub Actions updates config Configuration files updates deps Dependencies updates docs Documentation updates feat Feature security Security updates tests Tests updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: separate requirements specification from pyproject.toml

2 participants