qrcode_pix_ruby icon indicating copy to clipboard operation
qrcode_pix_ruby copied to clipboard

Add rjust to force 4 digit crc-16

Open everestcw opened this issue 3 years ago • 5 comments
trafficstars

We've noticed that in some cases when the Hex value of the CRC-16 starts with a 0 the conversion to string removes the zeros on the start. With this, we fix this problem. I've also added a taste to simulate that scenario.

everestcw avatar Sep 14 '22 20:09 everestcw

Thanks for creating a pull request! A maintainer will review your changes shortly.

mergeable[bot] avatar Sep 14 '22 20:09 mergeable[bot]

Fala Caio, @everestcw , tudo joia? 👋

Obrigado pela contribuição 🎉

pedrofurtado avatar Sep 16 '22 00:09 pedrofurtado

Codecov Report

Merging #27 (45ef3ce) into master (143c21c) will not change coverage. The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #27   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            4         4           
  Lines          209       215    +6     
=========================================
+ Hits           209       215    +6     
Impacted Files Coverage Δ
lib/qrcode_pix_ruby/payload.rb 100.00% <100.00%> (ø)
spec/qrcode_pix_ruby_spec.rb 100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

codecov-commenter avatar Sep 16 '22 00:09 codecov-commenter

@everestcw Caio, poderia por gentileza atualizar sua branch com a master e gerar uma entrada em CHANGELOG.md? Apos isso, acredito que o CI fique ok ✅

Fala, Pedro. Fiz os ajustes que você pediu. 👍

everestcw avatar Sep 17 '22 19:09 everestcw

@everestcw o CI falhou, provável q a alteração tenha quebrado algo no comportamento. Vale dar uma zoiadinha belê, rodar a suíte localmente qq coisa 🤝

pedrofurtado avatar Sep 19 '22 22:09 pedrofurtado