Skip to content

Refactor dependency groups and scope CI installs#262

Merged
freakboy3742 merged 3 commits into
beeware:mainfrom
gwdio:236-better-dependency-groups
May 19, 2026
Merged

Refactor dependency groups and scope CI installs#262
freakboy3742 merged 3 commits into
beeware:mainfrom
gwdio:236-better-dependency-groups

Conversation

@gwdio
Copy link
Copy Markdown
Contributor

@gwdio gwdio commented May 19, 2026

Refactors dependency groups in pyproject.toml to ensure that github workflow runs import the minimum amount of packages.

Groups have been updated to the following:

  • briefcase — provides cookiecutter; included by test and dev
  • pre-commit — used by the pre-commit CI job (--group pre-commit)
  • tox — used by the unit tests CI job (--group tox)
  • test — full test suite deps; includes briefcase
  • dev — local development convenience; includes all of the above

Afterwards, CI has been updated to use the new dependency groups.

This ensures that CI runs import dependency groups logically and do not overimport.

Refs #236

PR Checklist:

  • I will abide by the BeeWare Code of Conduct
  • I have read and have followed the CONTRIBUTING.md file
  • This PR was generated or assisted using an AI tool

Assisted-by: Claude Sonnet 4.6

Extract briefcase, pre-commit, and tox into their own dependency groups
so CI jobs install only what they need instead of the full dev group.
Copy link
Copy Markdown
Contributor

@kattni kattni left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! One suggestion in-line.

Comment thread pyproject.toml Outdated
Both link styles are equivalently valid.

Co-authored-by: Kattni <kattni@kattni.com>
Comment thread pyproject.toml Outdated
Copy link
Copy Markdown
Contributor Author

@gwdio gwdio left a comment

Choose a reason for hiding this comment

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

Should we also update requirements.txt to remove black then?

Copy link
Copy Markdown
Member

@freakboy3742 freakboy3742 left a comment

Choose a reason for hiding this comment

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

👍 No need to clean up requirements.txt - it's only going to be there for another couple of hours.

@freakboy3742 freakboy3742 merged commit 122909c into beeware:main May 19, 2026
26 checks passed
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.

3 participants