Skip to content
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] stock_packaging_calculator: black, isort, prettier #1914

Closed
wants to merge 1 commit into from

Conversation

bosd
Copy link
Contributor

@bosd bosd commented Jan 13, 2024

Fixes:
image

@bosd bosd changed the title [IMP] stock_packaging_calculator: black, isort, prettier [15.0][IMP] stock_packaging_calculator: black, isort, prettier Jan 13, 2024
@pedrobaeza pedrobaeza added this to the 15.0 milestone Jan 13, 2024
@pedrobaeza
Copy link
Member

This is not correct, as it's by design to put a non breakable space.

@bosd
Copy link
Contributor Author

bosd commented Jan 13, 2024

Previously flake8 and autoflake was broken on 15.0 branch.
After version bump of those modules it generated this error.

This is not correct, as it's by design to put a non breakable space.

Is a update of flake8 rules needed to exclude this E321?

Let's track it in: OCA/oca-addons-repo-template#243

@bosd
Copy link
Contributor Author

bosd commented Jan 13, 2024

Mentioned error is generated locally by
pre-commit run -a -v

When invoking
pre-commit run -v
The error is not shwowing. It is ok on CI here as well.
So closing this one.

@bosd bosd closed this Jan 13, 2024
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.

3 participants