Add black/flake8 dev deps and harden Jira Codex workflow#505
Add black/flake8 dev deps and harden Jira Codex workflow#505
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 10e0d55be3
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
Pull request overview
This PR adds Python formatting/linting tools to the project’s dev dependency set (and regenerates uv.lock accordingly), and updates the Jira Codex GitHub Actions workflow to validate tool availability with the new setup.
Changes:
- Add
blackandflake8to thedevdependency group inpyproject.toml. - Regenerate
uv.lockto includeblack,flake8, and their transitive dependencies. - Adjust the Jira Codex workflow’s “Verify tooling exists” step to run tool version checks via
uv.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
uv.lock |
Updated lockfile to include black, flake8, and related packages. |
pyproject.toml |
Adds black/flake8 to [dependency-groups].dev and reorders dev deps. |
.github/workflows/jira_codex_pr.yml |
Updates tool verification commands to use uv run --with …. |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
| "behave>=1.3.3", | ||
| "black>=25.9.0", | ||
| "faker>=25.0.0", | ||
| "flake8>=7.3.0", | ||
| "pyhamcrest>=2.0.3", | ||
| "pytest>=8.4.0", | ||
| "python-dotenv>=1.1.1", | ||
| "requests>=2.32.5", |
There was a problem hiding this comment.
The dev dependencies are not in alphabetical order. Currently: black, behave, flake8, faker, pyhamcrest, pytest, python-dotenv, requests.
Alphabetically they should be: behave, black, faker, flake8, pyhamcrest, pytest, python-dotenv, requests.
While this doesn't affect functionality, maintaining alphabetical order improves maintainability and reduces merge conflicts when adding new dependencies.
Summary
Validation