Skip to content

Add linting to the action#23

Merged
tjgurwara99 merged 4 commits into
mainfrom
tjgurwara99/add-linting-to-the-project
Jul 20, 2026
Merged

Add linting to the action#23
tjgurwara99 merged 4 commits into
mainfrom
tjgurwara99/add-linting-to-the-project

Conversation

@tjgurwara99

@tjgurwara99 tjgurwara99 commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

Closes https://github.com/github/code-scanning/issues/23716

This PR introduces linting in this repository so that the project follows python standards. We also introduce the usage of uv as a package manager, which would allow us to make sure that we have our dependencies in sync.

@tjgurwara99
tjgurwara99 marked this pull request as ready for review July 20, 2026 15:21
Copilot AI review requested due to automatic review settings July 20, 2026 15:21

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

This PR adds Python linting/format checking via Ruff and adopts uv (with pyproject.toml + uv.lock) to manage and lock development dependencies, while updating existing Python code/tests to satisfy the new linting standards.

Changes:

  • Introduces uv project configuration (pyproject.toml) and a lockfile (uv.lock) with Ruff as a dev dependency.
  • Adds a dedicated lint workflow and updates CI to install dependencies via uv before running unit tests.
  • Applies Ruff-driven formatting/typing cleanups across the Python source and unit tests.
Show a summary per file
File Description
uv.lock Adds uv lockfile to pin dev dependency resolution (Ruff).
pyproject.toml Defines the uv project and Ruff configuration (lint + formatting rules).
.github/workflows/lint.yml New workflow to run Ruff lint + format checks in CI.
.github/workflows/ci.yml Updates CI to install uv + dev dependencies and run tests via uv run.
upload_coverage.py Formatting/type-hint updates to comply with Ruff rules.
status_report.py Formatting/type-hint updates to comply with Ruff rules.
categorised_error.py Minor formatting fix to satisfy linting.
post_upload_coverage.py Minor formatting (blank line) to satisfy linting.
test_upload_coverage.py Formatting/modernization edits to satisfy linting.
test_status_report.py Formatting edits to satisfy linting.
test_post_upload_coverage.py Formatting edits to satisfy linting.
.gitignore Ignores .venv/ and .ruff_cache/ created by new tooling.

Review details

Tip

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

  • Files reviewed: 10/12 changed files
  • Comments generated: 3
  • Review effort level: Low

Comment thread .github/workflows/lint.yml
Comment thread .github/workflows/lint.yml Outdated
Comment thread .github/workflows/ci.yml Outdated
tjgurwara99 and others added 2 commits July 20, 2026 16:26
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@tjgurwara99
tjgurwara99 requested a review from jkbschmid July 20, 2026 15:28
@tjgurwara99
tjgurwara99 merged commit f8bb484 into main Jul 20, 2026
2 checks passed
@tjgurwara99
tjgurwara99 deleted the tjgurwara99/add-linting-to-the-project branch July 20, 2026 16:19
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