-
-
Notifications
You must be signed in to change notification settings - Fork 526
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
[15.0][IMP] Update pre commit #3875
base: 15.0
Are you sure you want to change the base?
Conversation
Hi @pedrobaeza, |
4324fde
to
12fd19d
Compare
This comment was marked as resolved.
This comment was marked as resolved.
@@ -285,7 +285,7 @@ def seur_atlas_cancel_shipment(self, pickings): | |||
|
|||
def seur_atlas_get_tracking_link(self, picking): | |||
return ( | |||
f"https://www.seur.com/livetracking/" | |||
f"https://www.seur.com/livetracking/" # NOQA E231 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Por qué esos comentarios NOQA
? No deberían hacer falta.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Con python3.12+ y la versión de flake8 actual en el repositorio no está solucionado. PyCQA/flake8#1845
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Entonces es un problema exclusivo de Python 3.12?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No puedes actualizar a pycodestyle 2.11.0?
Plantilla actualizada para que entre OCA/oca-addons-repo-template#243 y solventar error en flake8 con python3.10+