Skip to content

Fix lint#92

Merged
virtualdj merged 2 commits into
masterfrom
fix-lint2
Oct 2, 2025
Merged

Fix lint#92
virtualdj merged 2 commits into
masterfrom
fix-lint2

Conversation

@virtualdj

Copy link
Copy Markdown
Owner

Sistema l'errore che mandava in crisi ruff; l'else non gli piaceva col continue 🤷🏻‍♂️

@virtualdj virtualdj added patch New patch/hotfix/minor changes maintenance Applies to changes in actions or dependencies labels Oct 2, 2025
@moddroid94

moddroid94 commented Oct 2, 2025

Copy link
Copy Markdown
Contributor

Ciao,

credo che l'errore di ruff nei check sia dato dalla regola S320 che e' stata deprecata, va' rimossa da pyproject.toml

Per la modifica in teoria e' piu' corretto cosi', ma avevo paura che quel continue non skippasse l'intero loop ma solo l'if, else, quindi l'ho lasciato, guardando meglio nelle docs dovrebbe essere corretto cosi'.

Edit: Apposto hai gia' visto tutto 😂

@virtualdj

Copy link
Copy Markdown
Owner Author

Rimossa anche la regola S320 come da suggerimento.

@virtualdj virtualdj merged commit 1a8c525 into master Oct 2, 2025
2 of 3 checks passed
@virtualdj

Copy link
Copy Markdown
Owner Author

va' rimossa da pyproject.toml

L'ho rimossa ma non sono sicuro che ora passi correttamente, perché l'altra PR #91 mi dà ancora lo stesso errore.

@virtualdj virtualdj deleted the fix-lint2 branch October 2, 2025 19:57
@moddroid94

Copy link
Copy Markdown
Contributor

va' rimossa da pyproject.toml

L'ho rimossa ma non sono sicuro che ora passi correttamente, perché l'altra PR #91 mi dà ancora lo stesso errore.

credo che sia perche' ha runnato le actions prima del merge, forse dalle actions direttamente puoi provare a fargliela rifare, altrimenti devi rifare la PR per vederlo, credo 😅

@virtualdj

Copy link
Copy Markdown
Owner Author

Gliel'ho fatta rifare, dà lo stesso errore; sospetto a questo punto sia perché bisogna prima fare il rebase della PR altrimenti non vede il file aggiornato?

@moddroid94

Copy link
Copy Markdown
Contributor

Gliel'ho fatta rifare, dà lo stesso errore; sospetto a questo punto sia perché bisogna prima fare il rebase della PR altrimenti non vede il file aggiornato?

si probabile perche' vedevo che effettivamente non ha i change che hai fatto nell'ultimo merge in quel branch

@virtualdj

Copy link
Copy Markdown
Owner Author

Era quello, grazie! 🏆

@moddroid94

Copy link
Copy Markdown
Contributor

Figurati 😁

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Applies to changes in actions or dependencies patch New patch/hotfix/minor changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants