Skip to content

chore(docs): add required try block#2186

Merged
eddycharly merged 2 commits into
kyverno:mainfrom
noeleont:chore/fix-docs
Dec 5, 2024
Merged

chore(docs): add required try block#2186
eddycharly merged 2 commits into
kyverno:mainfrom
noeleont:chore/fix-docs

Conversation

@noeleont

@noeleont noeleont commented Dec 5, 2024

Copy link
Copy Markdown
Contributor

Explanation

I went trough the getting started and noticed a missing try block.

Related issue

Proposed Changes

Checklist

Further Comments

@noeleont
noeleont requested a review from a team as a code owner December 5, 2024 10:55
@eddycharly

Copy link
Copy Markdown
Member

Thanks for catching this and opening the fix PR!

@eddycharly eddycharly added this to the v0.2.13 milestone Dec 5, 2024
@eddycharly
eddycharly enabled auto-merge (squash) December 5, 2024 11:21
@eddycharly

Copy link
Copy Markdown
Member

@noeleont can you sign-off your commits please ?

auto-merge was automatically disabled December 5, 2024 14:41

Head branch was pushed to by a user without write access

@eddycharly
eddycharly enabled auto-merge (squash) December 5, 2024 14:43
@codecov

codecov Bot commented Dec 5, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 67.17%. Comparing base (86321cd) to head (e856d72).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2186      +/-   ##
==========================================
+ Coverage   67.12%   67.17%   +0.04%     
==========================================
  Files         138      138              
  Lines        6644     6650       +6     
==========================================
+ Hits         4460     4467       +7     
+ Misses       1859     1858       -1     
  Partials      325      325              

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

@eddycharly
eddycharly merged commit 006ea09 into kyverno:main Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants