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

cancel egci test case #637

Merged
merged 1 commit into from
Jul 28, 2023
Merged

cancel egci test case #637

merged 1 commit into from
Jul 28, 2023

Conversation

bzsuni
Copy link
Collaborator

@bzsuni bzsuni commented Jul 27, 2023

cancel egci test case

@bzsuni bzsuni added the release/none should not recoded in new release label Jul 27, 2023
@bzsuni bzsuni requested a review from weizhoublue as a code owner July 27, 2023 02:15
@codecov
Copy link

codecov bot commented Jul 27, 2023

Codecov Report

Merging #637 (0bb14b4) into main (4e92691) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #637   +/-   ##
=======================================
  Coverage   10.45%   10.45%           
=======================================
  Files          19       19           
  Lines        3864     3864           
=======================================
  Hits          404      404           
  Misses       3384     3384           
  Partials       76       76           
Flag Coverage Δ
unittests 10.45% <ø> (ø)

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

Copy link
Contributor

@biqiangwu biqiangwu left a comment

Choose a reason for hiding this comment

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

后面是不是可以不用大面积注释,加个 P 取消测试用例

@weizhoublue
Copy link
Collaborator

pending 不就好了么

@bzsuni
Copy link
Collaborator Author

bzsuni commented Jul 28, 2023

后面是不是可以不用大面积注释,加个 P 取消测试用例

crd 有更改,有些函数不适用了,编译会报错

@bzsuni
Copy link
Collaborator Author

bzsuni commented Jul 28, 2023

pending 不就好了么

后面crd有变更,有些函数不适用了,所以这里注释掉了

@weizhoublue weizhoublue merged commit c6fd7fa into main Jul 28, 2023
25 checks passed
@weizhoublue weizhoublue deleted the bz/e2e/cancelEgciTest branch July 28, 2023 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release/none should not recoded in new release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants