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.

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

Bump ruff from 0.7.2 to 0.7.3 #11222

Closed
wants to merge 1 commit into from
Closed

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Nov 8, 2024

Bumps ruff from 0.7.2 to 0.7.3.

Release notes

Sourced from ruff's releases.

0.7.3

Release Notes

Preview features

  • Formatter: Disallow single-line implicit concatenated strings (#13928)
  • [flake8-pyi] Include all Python file types for PYI006 and PYI066 (#14059)
  • [flake8-simplify] Implement split-of-static-string (SIM905) (#14008)
  • [refurb] Implement subclass-builtin (FURB189) (#14105)
  • [ruff] Improve diagnostic messages and docs (RUF031, RUF032, RUF034) (#14068)

Rule changes

  • Detect items that hash to same value in duplicate sets (B033, PLC0208) (#14064)
  • [eradicate] Better detection of IntelliJ language injection comments (ERA001) (#14094)
  • [flake8-pyi] Add autofix for docstring-in-stub (PYI021) (#14150)
  • [flake8-pyi] Update duplicate-literal-member (PYI062) to alawys provide an autofix (#14188)
  • [pyflakes] Detect items that hash to same value in duplicate dictionaries (F601) (#14065)
  • [ruff] Fix false positive for decorators (RUF028) (#14061)

Bug fixes

  • Avoid parsing joint rule codes as distinct codes in # noqa (#12809)
  • [eradicate] ignore # language= in commented-out-code rule (ERA001) (#14069)
  • [flake8-bugbear] - do not run mutable-argument-default on stubs (B006) (#14058)
  • [flake8-builtins] Skip lambda expressions in builtin-argument-shadowing (A002) (#14144)
  • [flake8-comprehension] Also remove trailing comma while fixing C409 and C419 (#14097)
  • [flake8-simplify] Allow open without context manager in return statement (SIM115) (#14066)
  • [pylint] Respect hash-equivalent literals in iteration-over-set (PLC0208) (#14063)
  • [pylint] Update known dunder methods for Python 3.13 (PLW3201) (#14146)
  • [pyupgrade] - ignore kwarg unpacking for UP044 (#14053)
  • [refurb] Parse more exotic decimal strings in verbose-decimal-constructor (FURB157) (#14098)

Documentation

  • Add links to missing related options within rule documentations (#13971)
  • Add rule short code to mkdocs tags to allow searching via rule codes (#14040)

Install ruff 0.7.3

Install prebuilt binaries via shell script

curl --proto '=https' --tlsv1.2 -LsSf https://github.com/astral-sh/ruff/releases/download/0.7.3/ruff-installer.sh | sh

Install prebuilt binaries via powershell script

powershell -ExecutionPolicy ByPass -c "irm https://github.com/astral-sh/ruff/releases/download/0.7.3/ruff-installer.ps1 | iex"
</tr></table> 

... (truncated)

Changelog

Sourced from ruff's changelog.

0.7.3

Preview features

  • Formatter: Disallow single-line implicit concatenated strings (#13928)
  • [flake8-pyi] Include all Python file types for PYI006 and PYI066 (#14059)
  • [flake8-simplify] Implement split-of-static-string (SIM905) (#14008)
  • [refurb] Implement subclass-builtin (FURB189) (#14105)
  • [ruff] Improve diagnostic messages and docs (RUF031, RUF032, RUF034) (#14068)

Rule changes

  • Detect items that hash to same value in duplicate sets (B033, PLC0208) (#14064)
  • [eradicate] Better detection of IntelliJ language injection comments (ERA001) (#14094)
  • [flake8-pyi] Add autofix for docstring-in-stub (PYI021) (#14150)
  • [flake8-pyi] Update duplicate-literal-member (PYI062) to alawys provide an autofix (#14188)
  • [pyflakes] Detect items that hash to same value in duplicate dictionaries (F601) (#14065)
  • [ruff] Fix false positive for decorators (RUF028) (#14061)

Bug fixes

  • Avoid parsing joint rule codes as distinct codes in # noqa (#12809)
  • [eradicate] ignore # language= in commented-out-code rule (ERA001) (#14069)
  • [flake8-bugbear] - do not run mutable-argument-default on stubs (B006) (#14058)
  • [flake8-builtins] Skip lambda expressions in builtin-argument-shadowing (A002) (#14144)
  • [flake8-comprehension] Also remove trailing comma while fixing C409 and C419 (#14097)
  • [flake8-simplify] Allow open without context manager in return statement (SIM115) (#14066)
  • [pylint] Respect hash-equivalent literals in iteration-over-set (PLC0208) (#14063)
  • [pylint] Update known dunder methods for Python 3.13 (PLW3201) (#14146)
  • [pyupgrade] - ignore kwarg unpacking for UP044 (#14053)
  • [refurb] Parse more exotic decimal strings in verbose-decimal-constructor (FURB157) (#14098)

Documentation

  • Add links to missing related options within rule documentations (#13971)
  • Add rule short code to mkdocs tags to allow searching via rule codes (#14040)
Commits
  • fbf140a Bump version to 0.7.3 (#14197)
  • 670f958 [red-knot] Fix intersection simplification for ~Any/~Unknown (#14195)
  • fed35a2 [red-knot] Fix is_assignable_to for unions (#14196)
  • d1ef418 Docs: tweak rules documentation (#14180)
  • 272d24b [flake8-pyi] Add a fix for duplicate-literal-member (#14188)
  • 2624249 [red-knot] Minor: fix Literal[True] <: int (#14177)
  • 4b08d17 [red-knot] Add a new Type::KnownInstanceType variant (#14155)
  • 5b6169b [red-knot] Minor fix in intersection type comment (#14176)
  • 2040e93 [flake8-logging-format] Fix invalid formatting value in docs of `logging-ex...
  • 794eb88 [flake8-bandit] Typo in docs suspicious-pickle-import (S403) (#14175)
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [ruff](https://github.com/astral-sh/ruff) from 0.7.2 to 0.7.3.
- [Release notes](https://github.com/astral-sh/ruff/releases)
- [Changelog](https://github.com/astral-sh/ruff/blob/main/CHANGELOG.md)
- [Commits](astral-sh/ruff@0.7.2...0.7.3)

---
updated-dependencies:
- dependency-name: ruff
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot added dependencies Pull requests that update a dependency file python Pull requests that update Python code labels Nov 8, 2024
Copy link

dryrunsecurity bot commented Nov 8, 2024

DryRun Security Summary

The code change in this GitHub Pull Request is a routine update to the version of the "ruff" linting tool dependency in the "requirements-lint.txt" file, which should be approved after reviewing the changelog and thoroughly testing the application.

Expand for full summary

Summary:

The code change in this GitHub Pull Request is a routine update to the version of the "ruff" linting tool dependency in the "requirements-lint.txt" file. Updating linting tool dependencies is generally a positive change, as it can include bug fixes, performance improvements, and the ability to detect additional security issues in the codebase.

However, it's important to review the changelog or release notes for the "ruff" version update to understand the specific changes and ensure that there are no known security vulnerabilities or breaking changes introduced in the new version. Additionally, it's a good practice to thoroughly test the application after such a dependency update to verify that the linting process and the application's functionality remain intact.

Overall, this code change seems routine and does not raise any immediate security concerns. As an application security engineer, I would recommend approving the Pull Request, provided that the necessary testing and review of the "ruff" version update have been completed.

Files Changed:

  • requirements-lint.txt: This file was updated to use version 0.7.3 of the "ruff" linting tool dependency, up from the previous version 0.7.2.

Code Analysis

We ran 9 analyzers against 1 file and 0 analyzers had findings. 9 analyzers had no findings.

Riskiness

🟢 Risk threshold not exceeded.

View PR in the DryRun Dashboard.

1 similar comment
Copy link

dryrunsecurity bot commented Nov 8, 2024

DryRun Security Summary

The code change in this GitHub Pull Request is a routine update to the version of the "ruff" linting tool dependency in the "requirements-lint.txt" file, which should be approved after reviewing the changelog and thoroughly testing the application.

Expand for full summary

Summary:

The code change in this GitHub Pull Request is a routine update to the version of the "ruff" linting tool dependency in the "requirements-lint.txt" file. Updating linting tool dependencies is generally a positive change, as it can include bug fixes, performance improvements, and the ability to detect additional security issues in the codebase.

However, it's important to review the changelog or release notes for the "ruff" version update to understand the specific changes and ensure that there are no known security vulnerabilities or breaking changes introduced in the new version. Additionally, it's a good practice to thoroughly test the application after such a dependency update to verify that the linting process and the application's functionality remain intact.

Overall, this code change seems routine and does not raise any immediate security concerns. As an application security engineer, I would recommend approving the Pull Request, provided that the necessary testing and review of the "ruff" version update have been completed.

Files Changed:

  • requirements-lint.txt: This file was updated to use version 0.7.3 of the "ruff" linting tool dependency, up from the previous version 0.7.2.

Code Analysis

We ran 9 analyzers against 1 file and 0 analyzers had findings. 9 analyzers had no findings.

Riskiness

🟢 Risk threshold not exceeded.

View PR in the DryRun Dashboard.

@manuel-sommer
Copy link
Contributor

I fixed the ruff linter and ruff update here: #11224

@manuel-sommer
Copy link
Contributor

This can be closed @mtesauro

@mtesauro
Copy link
Contributor

@dependabot rebase

Copy link
Contributor Author

dependabot bot commented on behalf of github Nov 12, 2024

Looks like ruff is up-to-date now, so this is no longer needed.

@dependabot dependabot bot closed this Nov 12, 2024
@dependabot dependabot bot deleted the dependabot/pip/dev/ruff-0.7.3 branch November 12, 2024 20:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file python Pull requests that update Python code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants