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

Adds the format to processos juridicos #41

Merged
merged 4 commits into from
May 13, 2019
Merged

Adds the format to processos juridicos #41

merged 4 commits into from
May 13, 2019

Conversation

felipediogo
Copy link
Contributor

Adds the format to processos juridicos based on the premise that they'll are all unified now.

font:
https://www.conjur.com.br/2009-jan-23/cnj-define-padrao-numeracao-processos-todos-tribunais

@felipediogo
Copy link
Contributor Author

talvez o nome da pasta deveria ser format-processo-juridico pra dar mais ênfase, mas não sei.

@codecov
Copy link

codecov bot commented Apr 1, 2019

Codecov Report

Merging #41 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #41   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          26     28    +2     
  Lines         220    235   +15     
  Branches       40     44    +4     
=====================================
+ Hits          220    235   +15
Impacted Files Coverage Δ
packages/format-processo/src/constants.ts 100% <100%> (ø)
packages/format-processo/src/index.ts 100% <100%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1b1db75...45aec67. Read the comment docs.

@codecov
Copy link

codecov bot commented Apr 1, 2019

Codecov Report

Merging #41 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #41   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          26     28    +2     
  Lines         220    240   +20     
  Branches       40     45    +5     
=====================================
+ Hits          220    240   +20
Impacted Files Coverage Δ
packages/format-processo-juridico/src/index.ts 100% <100%> (ø)
packages/is-valid-cnpj/src/index.ts 100% <100%> (ø) ⬆️
packages/is-valid-cpf/src/index.ts 100% <100%> (ø) ⬆️
packages/format-processo-juridico/src/constants.ts 100% <100%> (ø)
packages/format-boleto/src/index.ts 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1b1db75...0d60203. Read the comment docs.

@hyanmandian
Copy link
Member

@andreoav tem alguma sugestão quanto ao nome do pacote? pensei em format-processo-jurico... mas não sei se é um bom nome :(

@andreoav
Copy link
Member

andreoav commented Apr 5, 2019

Por mim tranquilo, na estrutura nova do #36 ficaria a pasta processo-juridico na pasta formatters

@felipediogo
Copy link
Contributor Author

tá, mas dai pra esse PR a mudança seria só trocar o nome da pasta então? e depois em outro PR no novo formato re-alocar pra pasta formatters certo @andreoav ?

@felipediogo
Copy link
Contributor Author

Mudei a pasta de format-processo, para format-processo-juridico, oq vocês acham? @hyanmandian @andreoav ?

@hyanmandian
Copy link
Member

Só corrige os conflitos e a gente mergeia :D

@hyanmandian hyanmandian merged commit 90ae10a into brazilian-utils:master May 13, 2019
fxamauri added a commit to fxamauri/brazilian-utils that referenced this pull request May 15, 2019
hyanmandian added a commit that referenced this pull request May 20, 2019
Adds validate cpf numbers with letters fix #39 #42  and format to processos juridicos #41 in refactoring as a single package with tree-shaking.
andreoav pushed a commit that referenced this pull request May 31, 2019
Adds the format to processos juridicos
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

3 participants