fix(templates): match 'uv add' order of lint fields in pyproject.toml
Metadata
Current evaluation
Merged a fix updating pyproject.toml field ordering in kubernetes and machine profiles to align with uv add, preventing unnecessary diffs. Approved by two reviewers and merged after CI validation.
Suggested action: —
No scores available.
Issue body
This PR fixes a tiny issue with the `kubernetes` and `machine` profiles. Currently, running `uv add <package>` in a charm project causes an unrelated field ordering change in `pyproject.toml`. That can result in unnecessary diffs, so I'm switching the profiles to the "correct" field ordering.
---
- [X] I've followed the [contribution guidelines](https://github.com/canonical/charmcraft/blob/main/CONTRIBUTING.md).
- [X] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/).
- [X] I've successfully run `make lint && make test`.
- [ ] ~I've added or updated any relevant documentation.~
- [ ] I've updated the relevant release notes.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged a fix updating pyproject.toml field ordering in kubernetes and machine profiles to align with uv add, preventing unnecessary diffs. Approved by two reviewers and merged after CI validation. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged an external contribution updating kubernetes and machine profile templates to align pyproject.toml field ordering with uv add, eliminating unnecessary diffs from unrelated reordering. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix aligning kubernetes and machine profile pyproject.toml field ordering with uv add. Prevents unnecessary diffs during package additions. Approved and merged. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #396 feat(lint): add uv lockfile linter | starbase | merged | Merged changes adding a lint-uv-lockfile make target to validate uv.lock against pyproject.toml. Approved by two reviewers and passed all CI checks. | |
| #2557 fix(templates): improve explanation of how to manage library dependencies | charmcraft | merged | Merged improvements to pyproject.toml dependency management documentation for kubernetes and machine profiles. Updates align with the new recommendation to distribute libraries as Python packages. Approved by two reviewers and passed CI checks. | |
| #2829 fix(test): add ops>=2.0.0 to uv constraint-dependencies | charmcraft | merged | Added ops>=2.0.0 to uv constraint-dependencies in pyproject.toml and the uv_project test fixture. Incorporated reviewer feedback to constrain pyyaml>=5 and fix formatting. Approved by three reviewers and merged. |