-
Notifications
You must be signed in to change notification settings - Fork 165
Fix link in README.md #2945
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
base: main
Are you sure you want to change the base?
Fix link in README.md #2945
Conversation
|
Thanks for the PR! |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2945 +/- ##
=======================================
Coverage 78.21% 78.22%
=======================================
Files 690 690
Lines 118750 118750
Branches 16680 16679 -1
=======================================
+ Hits 92881 92892 +11
+ Misses 24981 24968 -13
- Partials 888 890 +2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Hi -- we have a required CI job failing due to the license statement not being found in the PR description. If you agree, would you add the following statement to this description:
I can then rerun this job and get this merged. Thanks! |
|
Sure thing, I updated the PR description accordingly! |
|
/codebuild_run(9c92dfb) |
Hello team! I noticed the issue while investigating potential adoption. The porting guide refactor in #1463 missed this.
I also took the liberty of splitting the affected run-on sentence.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and the ISC license.