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

[REF] pre-commit l10n_br_base #749

Merged
merged 1 commit into from
Oct 31, 2019

Conversation

mileo
Copy link
Member

@mileo mileo commented Oct 30, 2019

Signed-off-by: Luis Felipe Mileo mileo@kmee.com.br

Signed-off-by: Luis Felipe Mileo <mileo@kmee.com.br>
@mileo
Copy link
Member Author

mileo commented Oct 30, 2019

Eu não sei o por que mas esse módulo ficou pra trás.

Recomendo a leitura do post: https://odoo-community.org/blog/the-oca-blog-1/post/black-isort-pre-commit-97

Isto esta valendo para v13 e fizemos o backport para v12.

Mudaram algumas regras, e algumas coisas que fazíamos muito na localização, não vamos mais poder usar.

@renatonlima
Copy link
Member

/ocabot merge

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 12.0-ocabot-merge-pr-749-by-renatonlima-bump-no, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Oct 31, 2019
Signed-off-by renatonlima
@OCA-git-bot OCA-git-bot merged commit 651d7e1 into OCA:12.0 Oct 31, 2019
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at daa7075. Thanks a lot for contributing to OCA. ❤️

@mileo mileo deleted the 12.0-oca-pre-commit-l10n_br_base branch February 28, 2021 01:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants