Skip to content
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

🐛 Use the proper repo for lombok. #2029

Merged
merged 1 commit into from
Jul 8, 2022
Merged

🐛 Use the proper repo for lombok. #2029

merged 1 commit into from
Jul 8, 2022

Conversation

wenottingham
Copy link
Contributor

What kind of change does this PR introduce?

This switches the repo scanned for lombok.

https://deps.dev/maven/org.projectlombok%3Alombok doesn't show a scorecard.

My hypothesis is that's because this is scanning
github.com/rzwitserloot/lombok
which redirects to github.com/projectlombok/lombok

Assumption is that just scanning the actual project repo will fix this.

What is the current behavior?

Scores are indexed by the old rzwitserloot repo, even though it's scanning the projectlombok one.

What is the new behavior (if this is a feature change)?**

Scores are under the repo actually scanned.

Which issue(s) this PR fixes

NONE

Special notes for your reviewer

Does this PR introduce a user-facing change?

NONE

@wenottingham wenottingham temporarily deployed to integration-test July 8, 2022 19:01 Inactive
@github-actions
Copy link

github-actions bot commented Jul 8, 2022

Integration tests success for
[c648408]
(https://github.com/ossf/scorecard/actions/runs/2638038367)

@codecov
Copy link

codecov bot commented Jul 8, 2022

Codecov Report

Merging #2029 (c648408) into main (f3e21fa) will increase coverage by 2.62%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2029      +/-   ##
==========================================
+ Coverage   50.56%   53.19%   +2.62%     
==========================================
  Files          87       87              
  Lines        7202     7202              
==========================================
+ Hits         3642     3831     +189     
+ Misses       3315     3119     -196     
- Partials      245      252       +7     

Copy link
Contributor

@azeemshaikh38 azeemshaikh38 left a comment

Choose a reason for hiding this comment

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

Thanks!

@naveensrinivasan naveensrinivasan enabled auto-merge (squash) July 8, 2022 22:56
@naveensrinivasan naveensrinivasan merged commit 48291a3 into ossf:main Jul 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants