Skip to content

Align python-best-practices scripts and references with SKILL.md#5

Open
Copilot wants to merge 3 commits intomasterfrom
copilot/align-python-scripts-with-skill-md
Open

Align python-best-practices scripts and references with SKILL.md#5
Copilot wants to merge 3 commits intomasterfrom
copilot/align-python-scripts-with-skill-md

Conversation

Copy link
Contributor

Copilot AI commented Mar 12, 2026

  • Update scripts/lint.sh: replace black --check with ruff format --check, replace mypy with ty check
  • Update scripts/setup-project.sh: replace manual scaffold with uv init --lib
  • Update references/pyproject-template.toml: remove black/mypy deps and tool sections; add ty>=0.0.21 dev dependency
  • Update SKILL.md: fix "(Black default)" note, update scripts/references tables to reflect new tools
Original prompt

This section details on the original issue you should resolve

<issue_title>python-best-practices references and scripts should be aligned with the updated SKILL.md</issue_title>
<issue_description>Currently the references and scripts implement outdated practices that are not aligned with the current version of SKILL.md.
For example, mypy is used for type checking, black for formatting, while we should be using ty check and ruff format.
Furthermore, the init script should use uv init --lib to instantiate a fresh project.</issue_description>

<agent_instructions>Keep the implementation light. Less code is better. Remove anything that is not strictly aligned with the SKILL.md.</agent_instructions>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: VMinB12 <93780551+VMinB12@users.noreply.github.com>
Copilot AI changed the title [WIP] [python-best-practices] Align references and scripts with updated SKILL.md Align python-best-practices scripts and references with SKILL.md Mar 12, 2026
Copy link
Contributor

@VMinB12 VMinB12 left a comment

Choose a reason for hiding this comment

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

We are

dev = [
"black>=24.0",
"ruff>=0.4",
"mypy>=1.10",
Copy link
Contributor

Choose a reason for hiding this comment

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

We are missing the ty 0.0.21 dev dependency.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added ty>=0.0.21 to the dev dependencies in d7b0bb5.

Co-authored-by: VMinB12 <93780551+VMinB12@users.noreply.github.com>
@VMinB12 VMinB12 marked this pull request as ready for review March 12, 2026 10:35
@VMinB12
Copy link
Contributor

VMinB12 commented Mar 12, 2026

@hugodendievelb12 Can you make sure all tech experts have write access?
Screenshot 2026-03-12 at 11 36 29

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.

python-best-practices references and scripts should be aligned with the updated SKILL.md

2 participants