Skip to content
This repository has been archived by the owner on Apr 4, 2024. It is now read-only.

Problem: cli keyring-backend not working correctly (fix #724) #725

Merged
merged 2 commits into from
Nov 12, 2021

Conversation

leejw51crypto
Copy link
Contributor

@leejw51crypto leejw51crypto commented Nov 6, 2021

in adding key,
config/client.toml backend is not used

  1. adding key <- config/client.toml not used
  2. list key <- config/client.toml used

@codecov
Copy link

codecov bot commented Nov 6, 2021

Codecov Report

Merging #725 (4c28789) into main (a874c1e) will increase coverage by 0.01%.
The diff coverage is 0.00%.

❗ Current head 4c28789 differs from pull request most recent head 39d0b30. Consider uploading reports for the commit 39d0b30 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##             main     #725      +/-   ##
==========================================
+ Coverage   56.56%   56.58%   +0.01%     
==========================================
  Files          64       64              
  Lines        5776     5774       -2     
==========================================
  Hits         3267     3267              
+ Misses       2309     2307       -2     
  Partials      200      200              
Impacted Files Coverage Δ
client/keys.go 0.00% <0.00%> (ø)

@lgtm-com
Copy link

lgtm-com bot commented Nov 6, 2021

This pull request introduces 1 alert when merging ca705eb967b521c6a6294a42c1b5426a5c1dd4bc into d82b5c2 - view on LGTM.com

new alerts:

  • 1 for Useless assignment to local variable

@leejw51crypto leejw51crypto force-pushed the feature/724 branch 2 times, most recently from 3ed0f52 to 2a04f2b Compare November 6, 2021 09:00
client/keys.go Outdated Show resolved Hide resolved
client/keys.go Outdated Show resolved Hide resolved
Copy link
Contributor

@fedekunze fedekunze left a comment

Choose a reason for hiding this comment

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

Thanks! LGTM, can you add a bug fix changelog entry too?

client/keys.go Outdated Show resolved Hide resolved
client/keys.go Outdated Show resolved Hide resolved
client/keys.go Outdated Show resolved Hide resolved
@fedekunze
Copy link
Contributor

@leejw51crypto bump

@leejw51crypto
Copy link
Contributor Author

i'll rebase and add to changelog

@leejw51crypto leejw51crypto force-pushed the feature/724 branch 2 times, most recently from 1393d5d to 31721da Compare November 12, 2021 02:33
…evmos#724)

refactoring

fix lint

refactoring

Apply suggestions from code review

add changelog
@leejw51crypto
Copy link
Contributor Author

done

@fedekunze fedekunze enabled auto-merge (squash) November 12, 2021 08:14
@fedekunze fedekunze merged commit 72c1098 into evmos:main Nov 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants