mais icon indicating copy to clipboard operation
mais copied to clipboard

Update de dependências e higiene do pacote.

Open pedrocava opened this issue 5 months ago • 9 comments

pedrocava avatar Jul 24 '25 05:07 pedrocava

Cava back in action! Quando for para revisar é só marcar.

rdahis avatar Jul 24 '25 10:07 rdahis

Pode corrigir esses problemas https://cran.r-project.org/web/checks/check_results_basedosdados.html nesse PR @pedrocava?

Sabe o motivo dessa mensagem? image

aspeddro avatar Sep 15 '25 22:09 aspeddro

Pode corrigir esses problemas https://cran.r-project.org/web/checks/check_results_basedosdados.html nesse PR @pedrocava?

Sabe o motivo essa mensagem? image

Consigo sim, o CRAN me contatou pedindo para resolver também. É só um probleminha de formatação da URL que deixamos nas docs.

pedrocava avatar Sep 16 '25 01:09 pedrocava

Pode corrigir esses problemas https://cran.r-project.org/web/checks/check_results_basedosdados.html nesse PR @pedrocava? Sabe o motivo essa mensagem? image

Consigo sim, o CRAN me contatou pedindo para resolver também. É só um probleminha de formatação da URL que deixamos nas docs.

Valeu @pedrocava @pedrocava Vc que tem feito o processo de submissão do pacote ao CRAN? Dada a ermegência, pensei com @aspeddro em fazer a submissão apenas com o problema relatado pelo CRAN e, na sequência, faremos um mapeamento de melhorias (atualizar dependências; atualizar métodos de consulta aos metadados...) para implementar com mais calma

folhesgabriel avatar Sep 16 '25 13:09 folhesgabriel

Dá para submeter hoje, as-is, e subimos outra versão depois resolvendo essas funções que batem na API de metadados. A gente mantém o PR aberto enquanto isso. Todo mundo de acordo?

pedrocava avatar Sep 16 '25 17:09 pedrocava

Concordo. Mantem esse aberto e abre um PR com as correções do cran https://github.com/basedosdados/sdk/pull/1768/commits/6fbc7b3b496a966d7330d1d9045f49c8a8673bcf

aspeddro avatar Sep 16 '25 17:09 aspeddro

Na verdade, a ideia era manter esse PR aqui mesmo. Ele já tem mudanças úteis e mudar apenas o que o CRAN pediu não passa no CMD check. Fechado, galera?

pedrocava avatar Sep 16 '25 18:09 pedrocava

Combinado!

folhesgabriel avatar Sep 16 '25 19:09 folhesgabriel

Dear maintainer,

thanks, package basedosdados_0.2.3.tar.gz is on its way to CRAN.

Best regards,
CRAN teams' auto-check service
Flavor: r-devel-linux-x86_64-debian-gcc, r-devel-windows-x86_64
Check: *, Result: OK

pedrocava avatar Sep 16 '25 23:09 pedrocava