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

Regra para th em portletCalendarEx #482

Merged
merged 1 commit into from
Jun 13, 2018

Conversation

idgserpro
Copy link
Member

Sobrescreve a regra adicionada pelo bootstrap de collective.upload. Ver plonegovbr/brasil.gov.agenda#82 (comment)

Boas práticas para o PR: Checklist

Foi aberta uma issue relativa a esse PR;

  • Sim
  • Não
  • Não se aplica

Foram adicionados testes unitários;

  • Sim
  • Não
  • Não se aplica

Foram adicionados testes robots;

  • Sim
  • Não
  • Não se aplica

Foram adicionados upgradeSteps;

  • Sim
  • Não
  • Não se aplica (upgradeStep já havia sido adicionado e release não foi lançado)

Foi adicionada a modificação no CHANGES.rst do pacote, sempre no topo do arquivo (ou seja, na primeira linha do release ainda não lançado), contendo o seu nome de usuário do github e a referência ao issue que você está tratando (não esqueça de colocar no fim do arquivo a url para a issue, use o padrão do próprio arquivo);

  • Sim
  • Não
  • Não se aplica

PR não contém assuntos diferentes. PR's devem ter um objetivo claro para facilitar o review. Ex: não junte, num mesmo PR, alterações de code-analysis e de implementação. Faça um PR de code-analysis e depois o da implementação em si;

  • Sim
  • Não
  • Não se aplica

Evite muitos commits pequenos de um mesmo assunto no PR, sempre que possível efetue rebase;

  • Sim
  • Não
  • Não se aplica

Copy link
Member

@hvelarde hvelarde left a comment

Choose a reason for hiding this comment

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

falta entrada no changelog.

Sobrescreve a regra adicionada pelo bootstrap de collective.upload. Assim,
força centralização do nome do mês no portlet de calendário estendido.

Ver plonegovbr/brasil.gov.agenda#82 (comment)
@idgserpro idgserpro force-pushed the issue-82-brasil-gov-agenda-layout-titulo-portlet branch from 2883c3f to 0fe6739 Compare June 12, 2018 17:19
@idgserpro
Copy link
Member Author

Achei que não era necessário pois isso é um complemento do relato plonegovbr/brasil.gov.agenda#82 e o release contendo a correção inicial dele ainda não havia sido lançado. De qualquer forma, adicionei como pedido.

@idgserpro idgserpro merged commit f793615 into 1.x Jun 13, 2018
@idgserpro idgserpro deleted the issue-82-brasil-gov-agenda-layout-titulo-portlet branch June 13, 2018 11:57
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.

None yet

2 participants