Skip to content

Conversation

ratancs
Copy link
Contributor

@ratancs ratancs commented Mar 22, 2025

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)?**

  • Tests for the changes have been added (for bug fixes/features)

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.)

Scorecard now supports configuration files ending in either `.yml` or `.yaml` 

@ratancs ratancs requested a review from a team as a code owner March 22, 2025 20:44
@ratancs ratancs requested review from justaugustus and spencerschrock and removed request for a team March 22, 2025 20:44
@spencerschrock
Copy link
Member

Thanks for the contribution, can you please fix DCO?
The failed check run has details on how to remediate:
https://github.com/ossf/scorecard/pull/4568/checks?check_run_id=39233419694

Copy link
Member

@spencerschrock spencerschrock left a 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

Ratan Kumar and others added 5 commits April 7, 2025 14:17
Signed-off-by: Ratan Kumar <ratan@Ratans-MacBook-Pro.local>
Signed-off-by: Ratan Kumar <ratan@Ratans-MacBook-Pro.local>
Signed-off-by: Ratan Kumar <ratan@Ratans-MacBook-Pro.local>
This partially reverts commit 4479ae6.

Signed-off-by: Spencer Schrock <sschrock@google.com>
Copy link
Member

@spencerschrock spencerschrock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@spencerschrock spencerschrock enabled auto-merge (squash) April 7, 2025 20:34
@spencerschrock spencerschrock changed the title ✨ adds yaml support ✨ adds support for configs ending in .yaml Apr 7, 2025
Copy link

codecov bot commented Apr 7, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.39%. Comparing base (353ed60) to head (2dee830).
Report is 149 commits behind head on main.

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:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@spencerschrock spencerschrock merged commit b373542 into ossf:main Apr 7, 2025
42 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Check for .yaml extension on config file, not only .yml extension
2 participants