Skip to content
This repository has been archived by the owner on May 31, 2024. It is now read-only.

Update workflow_execution_config #401

Merged
merged 2 commits into from
Apr 13, 2023
Merged

Update workflow_execution_config #401

merged 2 commits into from
Apr 13, 2023

Conversation

pingsutw
Copy link
Member

@pingsutw pingsutw commented Apr 9, 2023

TL;DR

Update the format of workflow-execution-config

Type

  • Bug Fix
  • Feature
  • Plugin

Are all requirements met?

  • Code completed
  • Smoke tested
  • Unit tests added
  • Code documentation added
  • Any pending items have an associated Issue

Complete description

^^^

Tracking Issue

NA

Follow-up issue

NA

pingsutw added 2 commits April 9, 2023 12:53
Signed-off-by: Kevin Su <pingsutw@apache.org>
Signed-off-by: Kevin Su <pingsutw@apache.org>
@codecov
Copy link

codecov bot commented Apr 9, 2023

Codecov Report

Merging #401 (d68fddc) into master (29da288) will increase coverage by 0.77%.
The diff coverage is n/a.

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

@@            Coverage Diff             @@
##           master     #401      +/-   ##
==========================================
+ Coverage   66.57%   67.35%   +0.77%     
==========================================
  Files         145      145              
  Lines        6271     5235    -1036     
==========================================
- Hits         4175     3526     -649     
+ Misses       1816     1429     -387     
  Partials      280      280              
Flag Coverage Δ
unittests ?

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

Impacted Files Coverage Δ
cmd/delete/matchable_workflow_execution_config.go 100.00% <ø> (ø)
cmd/update/matchable_workflow_execution_config.go 100.00% <ø> (ø)

... and 132 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@pingsutw pingsutw merged commit bd00706 into master Apr 13, 2023
@pingsutw pingsutw deleted the config-update branch April 13, 2023 16:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants