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 [email protected]

@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

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.28%. Comparing base (5bf94c1) to head (bf3a8dd).
⚠️ Report is 4178 commits behind head on main.

Additional details and impacted files
@@            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     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@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
@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