Skip to content

fix: do not lowercase address#870

Draft
izyak wants to merge 2 commits intomainfrom
fix/do-not-lowercase-address
Draft

fix: do not lowercase address#870
izyak wants to merge 2 commits intomainfrom
fix/do-not-lowercase-address

Conversation

@izyak
Copy link
Collaborator

@izyak izyak commented Jul 25, 2023

Closes #ISSUE-NUMBER

Checklist:

  • I have performed a self-review of my own code
  • I have documented my code in accordance with the documentation guidelines
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • I have run the unit tests
  • I only have one commit (if not, squash them into one commit).
  • I have a descriptive commit message that adheres to the commit message guidelines

Please review the CONTRIBUTING.md file for detailed contributing guidelines.

@codecov
Copy link

codecov bot commented Jul 25, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 36.43%. Comparing base (43e7ce3) to head (720f905).
Report is 4 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main     #870      +/-   ##
============================================
+ Coverage     36.39%   36.43%   +0.04%     
  Complexity      811      811              
============================================
  Files           125      125              
  Lines         12061    12061              
  Branches       1091     1091              
============================================
+ Hits           4389     4394       +5     
+ Misses         7332     7327       -5     
  Partials        340      340              
Files with missing lines Coverage Δ
.../java/foundation/icon/btp/bts/BTPTokenService.java 82.98% <100.00%> (ø)
...main/java/foundation/icon/btp/bts/BlacklistDB.java 74.46% <100.00%> (ø)

... and 1 file with indirect coverage changes

@izyak izyak marked this pull request as draft August 18, 2023 08:55
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.

1 participant