Skip to content
This repository has been archived by the owner on Jul 12, 2022. It is now read-only.

Verify Credencial of config.json #322

Merged

Conversation

viniciussousazup
Copy link
Contributor

@viniciussousazup viniciussousazup commented Jul 29, 2020

- What I did
if user add a input of type credencial on config.json that not exist on providers return err

- How to verify it

- Description for the changelog

@viniciussousazup viniciussousazup changed the title Improvement/set credential Verify Credencial of config.json Jul 29, 2020
@codecov-commenter
Copy link

codecov-commenter commented Jul 29, 2020

Codecov Report

Merging #322 into ritchie-2.0.0 will decrease coverage by 0.07%.
The diff coverage is n/a.

Impacted file tree graph

@@                Coverage Diff                @@
##           ritchie-2.0.0     #322      +/-   ##
=================================================
- Coverage          65.29%   65.21%   -0.08%     
=================================================
  Files                 78       78              
  Lines               2550     2550              
=================================================
- Hits                1665     1663       -2     
- Misses               723      725       +2     
  Partials             162      162              
Impacted Files Coverage Δ
pkg/credential/finder.go 68.42% <ø> (ø)
pkg/formula/watcher/watcher.go 70.58% <0.00%> (-5.89%) ⬇️

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 09a4993...6444072. Read the comment docs.

@viniciussousazup viniciussousazup self-assigned this Jul 29, 2020
@viniciussousazup viniciussousazup added 2.0.0 🔨 improvement Improvement in features labels Jul 29, 2020
@sandokandias sandokandias merged commit c05e32d into ZupIT:ritchie-2.0.0 Jul 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🔨 improvement Improvement in features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants