-
Notifications
You must be signed in to change notification settings - Fork 527
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
✨ adds support for configs ending in .yaml #4568
Conversation
Thanks for the contribution, can you please fix DCO? |
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 the test. Just a few small comments.
And you'll still need to fix DCO on the original commit (7857e04) before I can merge this
https://github.com/ossf/scorecard/pull/4568/checks?check_run_id=39323374807
Signed-off-by: Ratan Kumar <[email protected]>
Signed-off-by: Ratan Kumar <[email protected]>
Signed-off-by: Ratan Kumar <[email protected]>
This partially reverts commit 4479ae6. Signed-off-by: Spencer Schrock <[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.
Thank you!
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4568 +/- ##
==========================================
+ Coverage 66.80% 68.39% +1.59%
==========================================
Files 230 249 +19
Lines 16602 18829 +2227
==========================================
+ Hits 11091 12879 +1788
- Misses 4808 5097 +289
- Partials 703 853 +150 🚀 New features to boost your workflow:
|
What kind of change does this PR introduce?
Fixes #4552
What is the current behavior?
What is the new behavior (if this is a feature change)?**
Which issue(s) this PR fixes
#4552
Special notes for your reviewer
Does this PR introduce a user-facing change?
For user-facing changes, please add a concise, human-readable release note to
the
release-note
(In particular, describe what changes users might need to make in their
application as a result of this pull request.)