Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(aries_vcx): change log msg after fn arg rename #1086

Merged
merged 1 commit into from
Dec 6, 2023
Merged

Conversation

xprazak2
Copy link
Contributor

@xprazak2 xprazak2 commented Dec 6, 2023

Encryption envelope was refactored in #1070,
but one log message is out of sync with the new naming.

Encryption envelope was refactored in #1070,
but one log message is out of sync with the new naming.

Signed-off-by: Ondrej Prazak <ondrej.prazak@absa.africa>
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (8eed467) 0.05% compared to head (7ad654a) 0.05%.

❗ Current head 7ad654a differs from pull request most recent head e8bab03. Consider uploading reports for the commit e8bab03 to get more accurate results

Additional details and impacted files
@@          Coverage Diff          @@
##            main   #1086   +/-   ##
=====================================
  Coverage   0.05%   0.05%           
=====================================
  Files        471     471           
  Lines      24015   24015           
  Branches    4292    4292           
=====================================
  Hits          13      13           
  Misses     24001   24001           
  Partials       1       1           
Flag Coverage Δ
unittests-aries-vcx 0.05% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Patrik-Stas Patrik-Stas merged commit 14c3526 into main Dec 6, 2023
27 checks passed
@Patrik-Stas Patrik-Stas deleted the log-msg branch December 6, 2023 18:42
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.

3 participants