Skip to content

Use getArguments() instead of getParamValues() in CWE showcases#689

Merged
haeter525 merged 1 commit into
ev-flow:masterfrom
zinwang:use_getarguments_instead_of_getparamvalues
Sep 16, 2024
Merged

Use getArguments() instead of getParamValues() in CWE showcases#689
haeter525 merged 1 commit into
ev-flow:masterfrom
zinwang:use_getarguments_instead_of_getparamvalues

Conversation

@zinwang

@zinwang zinwang commented Sep 13, 2024

Copy link
Copy Markdown
Contributor

Refer to Issue #688

@zinwang zinwang self-assigned this Sep 13, 2024
@codecov

codecov Bot commented Sep 13, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.71%. Comparing base (6e7e4d1) to head (95d2991).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #689   +/-   ##
=======================================
  Coverage   78.71%   78.71%           
=======================================
  Files          71       71           
  Lines        5656     5656           
=======================================
  Hits         4452     4452           
  Misses       1204     1204           
Flag Coverage Δ
unittests 78.71% <ø> (ø)

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.

@zinwang zinwang requested a review from haeter525 September 13, 2024 11:23

@haeter525 haeter525 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. Thank @zinwang

@haeter525 haeter525 added the documentation Improvements or additions to documentation label Sep 16, 2024
@haeter525 haeter525 merged commit 847c813 into ev-flow:master Sep 16, 2024
@zinwang zinwang deleted the use_getarguments_instead_of_getparamvalues branch December 28, 2024 07:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants