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

[14.0][UPD] erpbrasil.base #3541

Closed
wants to merge 1 commit into from
Closed

Conversation

Tiago370
Copy link

Atualiza erpbrasil.base para versão mais recente.

@OCA-git-bot
Copy link
Contributor

Hi @AndreMarcos, @mbcosta, @gabrielcardoso21, @ygcarvalh, @lfdivino, @mileo, @marcelsavegnago, @rvalyi, @luismalta, @renatonlima,
some modules you are maintaining are being modified, check this out!

@rvalyi
Copy link
Member

rvalyi commented Dec 10, 2024

Tem algum bug super importante que isso resolve? Porque senao fica ruim atrelar a versão do erpbrasil.base aqui... (a gente fez uma vez por causa de um bug super critico).

Normalmente nos modulos OCA a boa pratica é de não especificar strictamente versões das dependencias Python (fora bug umportante) e la no requirements do seu projeto X, ai sim vc pode ser mais especifico.

cc @antoniospneto

@marcelsavegnago
Copy link
Member

@rvalyi tem razão.. ja seria bom deixar sem especificar a versao

@rvalyi
Copy link
Member

rvalyi commented Dec 10, 2024

1 ou 2 anos atrás botamos essa versão minima senao a NFe não ia de jeito nenhum porque a assinatura de algum metodo tinha mudado. A gente pode deixar essa versão minimal antiga ou então tirar a versão minima (ja que foi uns 2 anos já)

@Tiago370
Copy link
Author

@rvalyi , é um erro de validação apenas (IE do Distrito Federal). Acredito que seja do mesmo nível de importância de outras correções recentes que teve por lá, nenhum tipo de vulnerabilidade ou conflito. erpbrasil/erpbrasil.base#49

@rvalyi
Copy link
Member

rvalyi commented Dec 10, 2024

então por mim não vale de forçar a versão aqui. Senão os módulos OCA estariam sempre pedindo vc atualizar dodas dependências Python sempre... A boa pratica é que no seu projeto X vc tente botar sempre as versões mais atualizadas (depois de testar) assim como o Runboat/CI fazem. Nisso eu acho que pode fechar esse PR.

@mileo mileo closed this Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants