-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Fix check links errors, use latest version of the tool #5199
Conversation
b56e726
to
cd8c4f0
Compare
Codecov Report
@@ Coverage Diff @@
## main #5199 +/- ##
==========================================
- Coverage 90.35% 90.32% -0.04%
==========================================
Files 188 188
Lines 11057 11057
==========================================
- Hits 9991 9987 -4
- Misses 841 844 +3
- Partials 225 226 +1
Continue to review full report at Codecov.
|
|---------- |---------------------------------------------- |-------------------------------------------------------------------------------------------------------------------------- | | ||
| Hashing | ["hashing/fnv"](https://pkg.go.dev/hash/fnv) | The project adopted this as the default hashing method due to the efficiency and is reasonable for non cryptographic use | | ||
| Testing | Use `t.Parallel()` where possible | Enabling more test to be run in parallel will speed up the feedback process when working on the project. | | ||
| Scenario | Recommended | Rationale | |
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.
Random nits fixed in CONTRIBUTING.md to prove things work.
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.
This was intelij idea complaining that the table is not well formatted, so I pressed the button to fix that :)
Signed-off-by: Bogdan Drutu <[email protected]>
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.
Thanks for fixing this
@@ -41,8 +41,7 @@ reasonably fast reviews. | |||
|
|||
### When adding a new component | |||
|
|||
Consider submitting different PRs for (more details about adding new components |
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.
This is fixed because the new check actually works, see https://github.com/open-telemetry/opentelemetry-collector/runs/5999196053?check_suite_focus=true
…y#5199) * Fix check links errors, use latest version of the tool See https://github.com/open-telemetry/opentelemetry-collector/runs/5999091156?check_suite_focus=true Signed-off-by: Bogdan Drutu <[email protected]> * Fix broken links in CONTRIBUTING.md Signed-off-by: Bogdan Drutu <[email protected]>
See https://github.com/open-telemetry/opentelemetry-collector/runs/5999091156?check_suite_focus=true
Signed-off-by: Bogdan Drutu [email protected]