Skip to content

Conversation

@robertatakenaka
Copy link
Member

🎯 Descrição

Este PR agrupa correções estruturais nas coleções Venezuela (VEN), Revista Eletrônica (RVE) e Revista de Odontologia (RVO), além de atualizar as definições de documentos citáveis e dependências do projeto.

🛠️ Alterações Principais


📝 Notas de Versão (Changelog)

  • Dependências: Update xylose to 1.35.13.
  • Configurações: Adicionado suporte ao tipo de documento data-article.

✅ Checklist para o Revisor

  • Os testes unitários refletem a nova versão da Xylose?
  • A inclusão de data-article impacta o cálculo de impacto de coleções específicas?
  • As correções nas coleções VEN/RVE/RVO foram validadas com os dados de produção?

Copilot AI review requested due to automatic review settings February 3, 2026 20:41
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Este PR atualiza dependências e ajusta a lista de tipos de documento citáveis para contemplar novos formatos (ex.: data-article).

Changes:

  • Atualiza xylose para 1.35.13 em requirements.txt.
  • Inclui data-article em CITABLE_DOCUMENT_TYPES para que o tipo seja tratado como “citável”.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.

File Description
requirements.txt Bump de versão do xylose para endereçar issues mencionadas.
choices.py Amplia a lista de tipos de documentos citáveis, impactando flags/contagens de “citable docs”.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

])

CITABLE_DOCUMENT_TYPES = (
'data-article',
Copy link

Copilot AI Feb 3, 2026

Choose a reason for hiding this comment

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

CITABLE_DOCUMENT_TYPES uses u'...' literals throughout (Python 2.7 compatibility is declared in setup.py). Adding 'data-article' as a byte string introduces a mixed str/unicode tuple; make it a unicode literal for consistency and to avoid subtle coercion issues in Py2 code paths.

Suggested change
'data-article',
u'data-article',

Copilot uses AI. Check for mistakes.
@robertatakenaka robertatakenaka merged commit f874107 into scieloorg:master Feb 3, 2026
7 checks passed
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.

1 participant