herbsjs / buchu

Use Cases - Uniform, auditable and secure use case library
Other
24 stars 22 forks source link

feat(audittrail): now audit trail can be configured by mode or output type to ignore some fields #92

Closed jhomarolo closed 1 year ago

jhomarolo commented 1 year ago

Fixes and changes #90

Proposed Changes

  1. fix ci
  2. now audit trail can be configured by mode or output type to ignore some fields
  3. improve coverage

Readiness Checklist

Author/Contributor

Reviewing Maintainer

jhomarolo commented 1 year ago

Suggestions are welcome! 😃

codecov[bot] commented 1 year ago

Codecov Report

Merging #92 (8d3d295) into master (833b005) will increase coverage by 0.50%. The diff coverage is 100.00%.

:exclamation: Current head 8d3d295 differs from pull request most recent head c9db42d. Consider uploading reports for the commit c9db42d to get more accurate results

@@            Coverage Diff             @@
##           master      #92      +/-   ##
==========================================
+ Coverage   97.64%   98.14%   +0.50%     
==========================================
  Files           7        7              
  Lines         255      270      +15     
==========================================
+ Hits          249      265      +16     
+ Misses          6        5       -1     
Impacted Files Coverage Δ
src/request.js 100.00% <100.00%> (ø)
src/usecase.js 97.14% <100.00%> (+0.77%) :arrow_up:
src/results.js 94.73% <0.00%> (+2.63%) :arrow_up:

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

dalssoft commented 1 year ago

@jhomarolo, all fine to merge! great contribution!

could you open a issue (and a PR) to document this new behavior? tks

herbsjs-robot commented 1 year ago

:tada: This PR is included in version 2.1.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket: