Skip to content

lib: trocar declaração de 'var' para 'const' / 'let' #31

Description

@Thor99

Hoje no node-boleto temos muitas ocorrências de declarações de variáveis usando var. Tendo em vista que hoje a linguagem oferece formas mais otimizadas de declarar variáveis, seria interessante fazer a troca.

Dado que nossa pasta de lógica principal é a lib/, seria interessante trocar os var dos arquivos da pasta, para let / const, dependendo do caso.

Ocorrências:

https://github.com/pagarme/node-boleto/blob/master/lib/boleto.js#L7
https://github.com/pagarme/node-boleto/blob/master/lib/boleto.js#L9
https://github.com/pagarme/node-boleto/blob/master/lib/boleto.js#L10
https://github.com/pagarme/node-boleto/blob/master/lib/boleto.js#L11
https://github.com/pagarme/node-boleto/blob/master/lib/boleto.js#L12
https://github.com/pagarme/node-boleto/blob/master/lib/boleto.js#L13
https://github.com/pagarme/node-boleto/blob/master/lib/boleto.js#L24
https://github.com/pagarme/node-boleto/blob/master/lib/boleto.js#L46
https://github.com/pagarme/node-boleto/blob/master/lib/boleto.js#L70
https://github.com/pagarme/node-boleto/blob/master/lib/boleto.js#L72
https://github.com/pagarme/node-boleto/blob/master/lib/boleto.js#L76
https://github.com/pagarme/node-boleto/blob/master/lib/edi-helper.js#L1
https://github.com/pagarme/node-boleto/blob/master/lib/formatters.js#L20
https://github.com/pagarme/node-boleto/blob/master/lib/formatters.js#L21
https://github.com/pagarme/node-boleto/blob/master/lib/formatters.js#L23
https://github.com/pagarme/node-boleto/blob/master/lib/formatters.js#L25
https://github.com/pagarme/node-boleto/blob/master/lib/formatters.js#L41
https://github.com/pagarme/node-boleto/blob/master/lib/formatters.js#L42
https://github.com/pagarme/node-boleto/blob/master/lib/formatters.js#L44
https://github.com/pagarme/node-boleto/blob/master/lib/formatters.js#L45
https://github.com/pagarme/node-boleto/blob/master/lib/formatters.js#L57
https://github.com/pagarme/node-boleto/blob/master/lib/formatters.js#L65
https://github.com/pagarme/node-boleto/blob/master/lib/formatters.js#L66
https://github.com/pagarme/node-boleto/blob/master/lib/formatters.js#L68
https://github.com/pagarme/node-boleto/blob/master/lib/formatters.js#L69
https://github.com/pagarme/node-boleto/blob/master/lib/formatters.js#L70
https://github.com/pagarme/node-boleto/blob/master/lib/formatters.js#L71
https://github.com/pagarme/node-boleto/blob/master/lib/formatters.js#L78

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions