Skip to content

Update de dependências e higiene do pacote. #1768

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Update de dependências e higiene do pacote. #1768

wants to merge 11 commits into from

Conversation

pedrocava
Copy link
Collaborator

No description provided.

@rdahis
Copy link
Member

rdahis commented Jul 24, 2025

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

@aspeddro
Copy link
Collaborator

aspeddro commented Sep 15, 2025 •
edited
Loading

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

Sabe o motivo dessa mensagem?
image

@pedrocava
Copy link
Collaborator Author

pedrocava commented Sep 16, 2025 •
edited
Loading

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 pedrocava marked this pull request as ready for review September 16, 2025 02:50
aspeddro
aspeddro reviewed Sep 16, 2025
View reviewed changes
r-package/R/http-client.R
endpoint,
query = list()) {

base_url <- "https://basedosdados.org/api/3/action/bd_"
Copy link
Collaborator

@aspeddro aspeddro Sep 16, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Esse API não exite, certo @rdahis? Usamos https://backend.basedosdados.org/api/v1/graphql

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Certo! Essa era a api do ckan

@folhesgabriel
Copy link
Contributor

folhesgabriel commented Sep 16, 2025 •
edited
Loading

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

@pedrocava
Copy link
Collaborator Author

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?

@aspeddro
Copy link
Collaborator

Concordo. Mantem esse aberto e abre um PR com as correções do cran 6fbc7b3

@folhesgabriel folhesgabriel marked this pull request as draft September 16, 2025 17:25
@pedrocava
Copy link
Collaborator Author

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?

@folhesgabriel
Copy link
Contributor

Combinado!

@folhesgabriel folhesgabriel marked this pull request as ready for review September 16, 2025 19:49
@pedrocava
Copy link
Collaborator Author 

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aspeddro aspeddro aspeddro left review comments

@folhesgabriel folhesgabriel folhesgabriel left review comments

Assignees

@pedrocava pedrocava

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@pedrocava @rdahis @aspeddro @folhesgabriel