-
Notifications
You must be signed in to change notification settings - Fork 54
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
📖 Update .bingo/README.md #1186
Conversation
I have been bitten by this too many times (i.e. running `bingo` commands in the .bingo directory). I wanted to make sure no one else falls into the same confusion. Signed-off-by: Todd Short <tshort@redhat.com>
✅ Deploy Preview for olmv1 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1186 +/- ##
=======================================
Coverage 77.55% 77.55%
=======================================
Files 35 35
Lines 1916 1916
=======================================
Hits 1486 1486
Misses 296 296
Partials 134 134
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
1d64a4d
Bingo will overwrite this, which can cause issues. This reverts commit 1d64a4d.
I have been bitten by this too many times (i.e. running
bingo
commands in the .bingo directory). I wanted to make sure no one else falls into the same confusion.Description
Reviewer Checklist