Skip to content
New issue

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

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

Already on GitHub? # to your account

feat: add ruff 0.9.5, 0.9.6 and update watchfiles #1898

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fusion44
Copy link
Contributor

Contribution checklist (recommended but not always applicable/required):

  • There's an automated test for this change
  • Commit messages or code include references to related issues or PRs (including third parties)
  • Commit messages are conventional - examples from the log include "feat: add changelog files to fixup hook", "fix(contourpy): allow wheel usage", and "test: add sqlalchemy2 test"

@@ -6,7 +6,7 @@ authors = ["Your Name <you@example.com>"]

[tool.poetry.dependencies]
python = "^3.11"
ruff = "0.8.6"
ruff = "0.9.6"
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you update the poetry.lock as well?

@@ -11394,6 +11394,7 @@
"setuptools"
],
"loguru": [
"flit-core",
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you move this into a separate commit?

@@ -15055,6 +15056,9 @@
"orderedmultidict": [
"setuptools"
],
"orderly-set": [
"setuptools"
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you move this into a separate commit as well.

@@ -4058,6 +4078,7 @@ lib.composeManyExtensions [
let
# Watchfiles does not include Cargo.lock in tarball released on PyPi for versions up to 0.17.0
getRepoHash = version: {
"1.0.4" = "sha256-VoYV13GsTaAWoLcSfuadLR2l8Xbn0MEd/Uh9EP/DgjE=";
Copy link
Contributor

Choose a reason for hiding this comment

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

If it isn't directly related with ruff then it should go in a separate commit as well.

@@ -4079,6 +4100,7 @@ lib.composeManyExtensions [
sha256 = getRepoHash prev.watchfiles.version;

getCargoHash = version: {
"1.0.4" = "";
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is this empty?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants