Skip to content

fix: delete twice "-24" param and up ruff for tests it - #565

Merged
nicklafleur merged 1 commit into
boxed:mainfrom
even-even:fix/delete_twice_param_in_status_by_exit_code
Sep 8, 2026
Merged

nicklafleur merged 1 commit into
boxed:mainfrom
even-even:fix/delete_twice_param_in_status_by_exit_code

Conversation

@even-even

Copy link
Copy Markdown
Contributor

Hello

During the update of the ruff version, I found an error.

`ruff check...............................................................Failed

  • hook id: ruff-check
  • exit code: 1

F601 Dictionary key literal -24 repeated
--> src/mutmut/main.py:99:9
|
97 | 36: "timeout",
98 | 37: "caught by type check",
99 | -24: "timeout", # SIGXCPU
| ^^^
100 | 24: "timeout", # SIGXCPU
101 | 152: "timeout", # SIGXCPU
|
help: Remove repeated key literal -24

Found 1 error.`

For fix I removed twice param

Also: up ruff version for tests this

@nicklafleur

Copy link
Copy Markdown
Collaborator

I believe I fixed this with an unrelated merge, but feel free to recheck

@even-even

Copy link
Copy Markdown
Contributor Author

This branch has conflicts that must be resolved

Changes can be cleanly merged.

  • src/mutmut/main.py

The error is being reproduced again...

`ruff check...............................................................Failed

  • hook id: ruff-check
  • exit code: 1

F601 Dictionary key literal -24 repeated
--> src/mutmut/stats.py:28:9
|
26 | 36: "timeout",
27 | 37: "caught by type check",
28 | -24: "timeout", # SIGXCPU
| ^^^
29 | 24: "timeout", # SIGXCPU
30 | 152: "timeout", # SIGXCPU
|
help: Remove repeated key literal -24

Found 1 error.`

https://docs.python.org/3/reference/expressions.html#dictionary-displays
Restrictions on the types of the key values are listed earlier in section [The standard type hierarchy](https://docs.python.org/3/reference/datamodel.html#types). (To summarize, the key type should be [hashable](https://docs.python.org/3/glossary.html#term-hashable), which excludes all mutable objects.) Clashes between duplicate keys are not detected; the last value (textually rightmost in the display) stored for a given key value prevails.

That is, the first value won’t be taken into account here.

@even-even
even-even force-pushed the fix/delete_twice_param_in_status_by_exit_code branch from f06922d to 169240e Compare September 8, 2026 05:39
@nicklafleur
nicklafleur merged commit 630e846 into boxed:main Sep 8, 2026
7 checks passed
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.

2 participants