search

mincong-h / finance-toolkit

Finance Toolkit
3 stars 1 forks source link

Allow specifying extra patterns for Revolut #81

Closed mincong-h closed 2 years ago

mincong-h commented 2 years ago

because I realized that the existing patterns weren't defined correctly. This will still happen in the future, which may cause by: bugs, renaming from the bank, ... So it makes sense to allow specifying extra patterns by user without needing the modify the source code.

codecov-commenter commented 2 years ago

Codecov Report

Merging #81 (1d25876) into master (48458eb) will increase coverage by 0.02%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #81      +/-   ##
==========================================
+ Coverage   94.23%   94.26%   +0.02%     
==========================================
  Files          11       11              
  Lines         607      610       +3     
  Branches       97       98       +1     
==========================================
+ Hits          572      575       +3     
  Misses         20       20              
  Partials       15       15
Impacted Files Coverage Δ
finance_toolkit/tx.py 97.03% <ø> (ø)
finance_toolkit/revolut.py 85.41% <100.00%> (+0.97%) :arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more