Skip to content

[project] br_sou_da_paz_armas_municoes#1481

Draft
tricktx wants to merge 15 commits into
mainfrom
br_sou_da_paz_armas_municoes
Draft

[project] br_sou_da_paz_armas_municoes#1481
tricktx wants to merge 15 commits into
mainfrom
br_sou_da_paz_armas_municoes

Conversation

@tricktx
Copy link
Copy Markdown
Contributor

@tricktx tricktx commented Mar 27, 2026

Descrição do PR:

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 27, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 42479ca8-e32a-4ffb-bdaa-f611bb511bc6

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch br_sou_da_paz_armas_municoes

Comment @coderabbitai help to get the list of available commands and usage tips.

tricktx and others added 6 commits March 29, 2026 20:43
Replace the `folder` parameter in `tb.create()` with
`bucket_name` and `mode` parameters on the `bd.Table` constructor,
aligning with the `basedosdados` v2.1.0b1 API. Update the README
to document the new usage pattern.
dataset_id="br_sou_da_paz_dataset-name",
table_id="table-name",
bucket_name="basedosdados-consultoria", # Nome do bucket
mode="sou_da_paz", # Nome da pasta no bucket, deve ser o nome da organização
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

O nome do parâmetro mode parece não fazer sentido, mas mantive ele para reduzir as alterações no pacote em basedosdados/sdk#1784. No pacote da BD mode representa duas coisas ao mesmo tempo e isso fica confuso.

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.

2 participants