Skip to content

Conversation

@remzouille
Copy link
Contributor

No description provided.

@f-lawe
Copy link
Owner

f-lawe commented May 13, 2025

Ca fait pas mal de questions/remarques, je te laisse me dire ce que tu en penses !

En parallèle, est-ce que tu as installé toutes les dépendances du projet dans requirements.txt, en particulier ruff ?

@remzouille
Copy link
Contributor Author

J'utilise ruff, mais pas toujours. Il n'y a pas de problèmes bien graves signalés, à part des lignes trop longues mais c'était déjà le cas avec les versions précédentes.

@f-lawe
Copy link
Owner

f-lawe commented May 14, 2025

Tu as quand même moyen de corriger ça ? Pas d'urgence évidemment. Je vais regarder si je ne peux pas le faire également de mon côté, je ne sais pas comment ça peut marcher avec un fork (ça touche aux limites de mes habitudes avec git).

@remzouille
Copy link
Contributor Author

C'est corrigé. Il ne reste qu'une erreur sur le bloc des imports, mais les classer différemment fait perdre en lisibilité.

Je ne suis pas non plus un expert en git, mais tu devrais pouvoir créer une nouvelle branche et tirer la PR dans cette branche.

https://docs.github.com/fr/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

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