Skip to content

Document ruff format check in linting guidelines#44

Merged
daviddavis merged 2 commits into
pulp:mainfrom
daviddavis:update-linting-docs
Jul 17, 2026
Merged

Document ruff format check in linting guidelines#44
daviddavis merged 2 commits into
pulp:mainfrom
daviddavis:update-linting-docs

Conversation

@daviddavis

Copy link
Copy Markdown
Collaborator

📜 Checklist

  • Commits are cleanly separated with meaningful messages (simple features and bug fixes should be squashed to one commit)
  • A changelog entry or entries has been added for any significant changes
  • Follows the Pulp policy on AI Usage
  • (For new features) - User documentation and test coverage has been added

See: Pull Request Walkthrough

Copilot AI review requested due to automatic review settings July 17, 2026 12:54

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the contributor/agent workflow documentation to include Ruff formatting verification alongside lint checks, aligning local pre-commit steps with formatting expectations.

Changes:

  • Expands the linting section to instruct running multiple checks before committing.
  • Adds ruff format --check --diff . (with a note to use ruff format . to auto-fix) in addition to ruff check ..

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

@daviddavis
daviddavis enabled auto-merge (squash) July 17, 2026 12:56
Assisted-by: GitHub Copilot (Claude Opus 4.8)
@daviddavis
daviddavis force-pushed the update-linting-docs branch from 4de4c35 to 1a4c3e8 Compare July 17, 2026 12:58
Copilot AI review requested due to automatic review settings July 17, 2026 17:10
@daviddavis
daviddavis merged commit 7a6fb11 into pulp:main Jul 17, 2026
9 checks passed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants