Skip to content

chore: add local shellcheck tools#521

Merged
arkodg merged 1 commit intoenvoyproxy:mainfrom
Xunzhuo:chore-shellcheck
Oct 13, 2022
Merged

chore: add local shellcheck tools#521
arkodg merged 1 commit intoenvoyproxy:mainfrom
Xunzhuo:chore-shellcheck

Conversation

@Xunzhuo
Copy link
Copy Markdown
Member

@Xunzhuo Xunzhuo commented Oct 8, 2022

Resolve: #314

Signed-off-by: bitliu bitliu@tencent.com

@Xunzhuo Xunzhuo requested a review from a team as a code owner October 8, 2022 04:43
@Xunzhuo Xunzhuo marked this pull request as draft October 8, 2022 04:48
@Xunzhuo Xunzhuo force-pushed the chore-shellcheck branch 5 times, most recently from 91c1e8d to 5efdc93 Compare October 8, 2022 05:44
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Oct 8, 2022

Codecov Report

Merging #521 (bf3a8dd) into main (5bf94c1) will decrease coverage by 3.19%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #521      +/-   ##
==========================================
- Coverage   63.48%   60.28%   -3.20%     
==========================================
  Files          40       45       +5     
  Lines        4425     5564    +1139     
==========================================
+ Hits         2809     3354     +545     
- Misses       1471     2003     +532     
- Partials      145      207      +62     
Impacted Files Coverage Δ
internal/xds/translator/runner/runner.go 0.00% <0.00%> (-86.67%) ⬇️
internal/message/types.go 75.00% <0.00%> (-25.00%) ⬇️
internal/provider/kubernetes/gateway.go 51.85% <0.00%> (-21.25%) ⬇️
internal/provider/kubernetes/kubernetes.go 48.00% <0.00%> (-9.50%) ⬇️
internal/gatewayapi/translator.go 85.52% <0.00%> (-5.60%) ⬇️
internal/ir/xds.go 81.86% <0.00%> (-3.85%) ⬇️
internal/provider/kubernetes/httproute.go 59.13% <0.00%> (-3.71%) ⬇️
internal/xds/translator/translator.go 72.04% <0.00%> (-2.65%) ⬇️
internal/provider/kubernetes/gatewayclass.go 73.18% <0.00%> (-2.18%) ⬇️
internal/xds/translator/listener.go 85.81% <0.00%> (-0.71%) ⬇️
... and 13 more

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

@Xunzhuo Xunzhuo marked this pull request as ready for review October 8, 2022 05:50
Copy link
Copy Markdown
Contributor

@danehans danehans left a comment

Choose a reason for hiding this comment

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

LGTM. @arkodg @lizan WDYT?

@Xunzhuo Xunzhuo requested a review from arkodg October 11, 2022 02:38
Signed-off-by: bitliu <bitliu@tencent.com>
@Xunzhuo Xunzhuo requested review from LukeShu and arkodg and removed request for LukeShu October 12, 2022 04:00
@Xunzhuo
Copy link
Copy Markdown
Member Author

Xunzhuo commented Oct 12, 2022

Thanks @LukeShu, updated based on your comments.

@Xunzhuo Xunzhuo requested review from LukeShu and removed request for arkodg October 12, 2022 04:00
@danehans danehans added the area/ci CI and build related issues label Oct 12, 2022
@danehans danehans added this to the 0.2.0 milestone Oct 12, 2022
Copy link
Copy Markdown
Contributor

@LukeShu LukeShu left a comment

Choose a reason for hiding this comment

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

Thanks!

@arkodg arkodg merged commit 0be1aef into envoyproxy:main Oct 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ci CI and build related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Shellcheck Linter

6 participants