Skip to content

chore: add finos/git-proxy to the project list - #140

Merged
justaugustus merged 1 commit into
ossf:mainfrom
jescalada:add-finos-git-proxy-to-projects
Oct 15, 2026
Merged

justaugustus merged 1 commit into
ossf:mainfrom
jescalada:add-finos-git-proxy-to-projects

Conversation

@jescalada

@jescalada jescalada commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

What this changes

Adds finos/git-proxy to the project list. Plenty of other popular FINOS projects are already in the list, and GitProxy already makes use of OSSF scorecard (currently with a score of 8.7). The problem is that LFX Insights fetches the score based on the weekly cron job rather than the API, thus GitProxy appears unindexed:

image

Related

How it was tested

Checklist

  • Commits are signed off (git commit -s)
  • make build and make test pass
  • make lint is clean
  • Specs/docs updated if behavior changed
  • No employer/internal references added

If this PR touches the batch pipeline (cron/):

  • If cron/internal/format changed, the corresponding engine change exists
    upstream and schema_gen_test.go passes

If this PR touches the scan inventories (cron/internal/data/*.csv):

  • make add-projects was run and its output committed
  • make validate-projects passes

Signed-off-by: Juan Escalada <juanescalada175@gmail.com>
@jescalada

Copy link
Copy Markdown
Contributor Author

Just finished verifying that all checks (make build, make test, make lint and make validate-projects) are passing.

@codecov

codecov Bot commented Oct 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 49.49%. Comparing base (e435c38) to head (40289ae).
⚠️ Report is 257 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #140      +/-   ##
==========================================
+ Coverage   42.71%   49.49%   +6.78%     
==========================================
  Files          48       60      +12     
  Lines        2568     3291     +723     
==========================================
+ Hits         1097     1629     +532     
- Misses       1383     1504     +121     
- Partials       88      158      +70     
Components Coverage Δ
api 59.70% <57.70%> (∅)
cron 33.48% <82.51%> (+6.44%) ⬆️
core 68.55% <ø> (ø)
🚀 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.

@justaugustus justaugustus left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@jescalada — Thanks for the submission and the additional context on LFX Insights!

@justaugustus
justaugustus merged commit d68c847 into ossf:main Oct 15, 2026
23 checks passed
@jescalada

Copy link
Copy Markdown
Contributor Author

@justaugustus Thanks for the quick review! 🙂

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.

2 participants