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

394 - Converter data no formato brasileiro para uma representação textual #415

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

BeneBr
Copy link

@BeneBr BeneBr commented Sep 20, 2024

Descrição

Possibilitar aos usuários uma forma de ter uma representação textual de uma data no formato brasileito (dd/mm/aaa)

Mudanças Propostas

  1. Adicionada uma nova dependência num2words no qual transforma um determinado número em uma representação textual em vários idiomas;

  2. Adicionada uma free function convert_date_to_text onde os usuários podem passar uma string no formato dd/mm/aaaa e obter uma representação textual;

Checklist de Revisão

  • Eu li o Contributing.md
  • Os testes foram adicionados ou atualizados para refletir as mudanças (se aplicável).
  • Foi adicionada uma entrada no changelog / Meu PR não necessita de uma nova entrada no changelog.
  • A documentação em português foi atualizada ou criada, se necessário.
  • Se feita a documentação, a atualização do arquivo em inglês.
  • Eu documentei as minhas mudanças no código, adicionando docstrings e comentários. Instruções
  • O código segue as diretrizes de estilo e padrões de codificação do projeto.
  • Todos os testes passam. Instruções
  • O Pull Request foi testado localmente. Instruções
  • Não há conflitos de mesclagem.

Comentários Adicionais (opcional)

Issue Relacionada

Closes #394

Copy link

codecov bot commented Sep 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.81%. Comparing base (3f0d528) to head (f3da444).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #415      +/-   ##
==========================================
+ Coverage   99.78%   99.81%   +0.02%     
==========================================
  Files          18       20       +2     
  Lines         472      542      +70     
==========================================
+ Hits          471      541      +70     
  Misses          1        1              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Converts a given date in brazilian format to its textual
representation.

394
Comment on lines 77 to 80
def testMonthEnum(self):
for day, month in self.months_year:
mont = MonthsEnum(day)
self.assertEqual(mont.mont_name, month)
Copy link
Author

Choose a reason for hiding this comment

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

Suggested change
def testMonthEnum(self):
for day, month in self.months_year:
mont = MonthsEnum(day)
self.assertEqual(mont.mont_name, month)
def testMonthEnum(self):
for mont_number, month in self.months_year:
mont = MonthsEnum(month_number)
self.assertEqual(mont.mont_name, month)

@@ -98,7 +98,7 @@ emulate bash -c '. .../bin/activate'
Para testar se o ambiente virtual está ativo corretamente, execute o comando e verifique se a resposta é algo parecido com a seguinte:

```sh
$ poetry env inf
$ poetry env info
Copy link
Member

Choose a reason for hiding this comment

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

Boaa!! Obrigada pelos olhos de lince!! 👀

README.md Outdated
Exemplo:

````python
>>> from brutils.date import convert_date_to_text
Copy link
Member

Choose a reason for hiding this comment

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

Só para entrar no padrão dos outros imports :)

Suggested change
>>> from brutils.date import convert_date_to_text
>>> from brutils import convert_date_to_text

README_EN.md Outdated
Example:

````python
>>> from brutils.date import convert_date_to_text
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
>>> from brutils.date import convert_date_to_text
>>> from brutils import convert_date_to_text

DEZEMBRO = 12

@property
def mont_name(self) -> str:
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
def mont_name(self) -> str:
def month_name(self) -> str:

brutils/date.py Outdated
date_string = (
day_string.capitalize()
+ " de "
+ month.mont_name
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
+ month.mont_name
+ month.month_name

Comment on lines 77 to 80
def testMonthEnum(self):
for day, month in self.months_year:
mont = MonthsEnum(day)
self.assertEqual(mont.mont_name, month)
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
def testMonthEnum(self):
for day, month in self.months_year:
mont = MonthsEnum(day)
self.assertEqual(mont.mont_name, month)
def testMonthEnum(self):
for day, month in self.months_year:
month = MonthsEnum(day)
self.assertEqual(month.month_name, month)

Copy link
Author

Choose a reason for hiding this comment

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

Na verdade os nomes do teste em si estavam errados, acabei de alterá-los para isso:

def testMonthEnum(self):
        for number_month,  name_month in self.months_year:
            month = MonthsEnum(number_month)
            self.assertEqual(month.month_name, name_month)

Copy link
Member

@antoniamaia antoniamaia left a comment

Choose a reason for hiding this comment

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

Boa, @BeneBr, mandaram ver!! Obrigada pela contribuição, ajustei apenas alguns detalhes, qualquer coisa só chamar a gente!! 🥇

@BeneBr BeneBr requested a review from antoniamaia October 17, 2024 01:35
@BeneBr
Copy link
Author

BeneBr commented Oct 17, 2024

Assim que obtiver o approve, faço squash dos commits.

@BeneBr
Copy link
Author

BeneBr commented Dec 6, 2024

Alguém poderia aprovar para eu poder fazer o merge?

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.

Conversão de Data para Texto
2 participants