Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Address deploy-agent lint errors #1642

Merged
merged 3 commits into from
Jun 3, 2024
Merged

Address deploy-agent lint errors #1642

merged 3 commits into from
Jun 3, 2024

Commits on Jun 3, 2024

  1. Address deploy-agent lint errors 

    `ruff` was recently introduced to the precommit checks.

This PR addresses the existing checks for deploy-agent:

* Add `ruff` command to `tox.ini`
* Run `ruff check --fix`
* Run `ruff check --fix --unsafe-fixes`
  * Verified the fixes were safe
* Use explicit exceptions
* Remove unused variables and methods
* Remove trailing ws
    @osoriano
    osoriano committed Jun 3, 2024
    Configuration menu
    Copy the full SHA
    83f6c00 View commit details
    Browse the repository at this point in the history

  2. Remove ruff format

    @osoriano
    osoriano committed Jun 3, 2024
    Configuration menu
    Copy the full SHA
    16e9b0c View commit details
    Browse the repository at this point in the history

  3. Also remove djlint-format

    @osoriano
    osoriano committed Jun 3, 2024
    Configuration menu
    Copy the full SHA
    70817af View commit details
    Browse the repository at this point in the history