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

🌱 GitLab: Add log message #3319

Merged
merged 3 commits into from
Jul 28, 2023
Merged

🌱 GitLab: Add log message #3319

merged 3 commits into from
Jul 28, 2023

Conversation

raghavkaul
Copy link
Contributor

What kind of change does this PR introduce?

Adds log message when GitLab is running on non-HEAD that scores may be inaccurate.

Which issue(s) this PR fixes

Temporary warning #3193 to unblock release.

@raghavkaul raghavkaul temporarily deployed to gitlab July 26, 2023 19:34 — with GitHub Actions Inactive
@raghavkaul raghavkaul temporarily deployed to integration-test July 26, 2023 19:35 — with GitHub Actions Inactive
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.

Is this something we want in the results? Or just when running the CLI? If it's the latter, we can probably do this in the CLI somewhere and take it out easily once the issue is fixed.

clients/repo_client.go Outdated Show resolved Hide resolved
clients/repo_client.go Outdated Show resolved Hide resolved
Signed-off-by: Raghav Kaul <[email protected]>
@raghavkaul raghavkaul temporarily deployed to gitlab July 26, 2023 21:02 — with GitHub Actions Inactive
@raghavkaul raghavkaul temporarily deployed to integration-test July 26, 2023 21:02 — with GitHub Actions Inactive
@codecov
Copy link

codecov bot commented Jul 26, 2023

Codecov Report

Merging #3319 (c21ccc7) into main (b992be3) will decrease coverage by 11.37%.
The diff coverage is 0.00%.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #3319       +/-   ##
===========================================
- Coverage   72.83%   61.46%   -11.37%     
===========================================
  Files         173      173               
  Lines       12686    12694        +8     
===========================================
- Hits         9240     7803     -1437     
- Misses       2941     4464     +1523     
+ Partials      505      427       -78     

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.

Can we add a TODO comment with tracking issue?

Note: this can run multiple times, but not sure pushing the error deeper can be done as easily as right here

SCORECARD_EXPERIMENTAL=1 go run main.go --repo gitlab.com/ossf-test/scorecard-archived-project --commit 240e1fbc4c4ea614527c59f453bf54107d8833f2
...
Scorecard may be missing merge requests when running on non-HEAD of a GitLab repo. Code-Review scores may be lower.
Scorecard may be missing merge requests when running on non-HEAD of a GitLab repo. Code-Review scores may be lower.
Scorecard may be missing merge requests when running on non-HEAD of a GitLab repo. Code-Review scores may be lower.
Scorecard may be missing merge requests when running on non-HEAD of a GitLab repo. Code-Review scores may be lower.
...

@raghavkaul raghavkaul temporarily deployed to gitlab July 27, 2023 22:35 — with GitHub Actions Inactive
@raghavkaul raghavkaul temporarily deployed to integration-test July 27, 2023 22:35 — with GitHub Actions Inactive
@raghavkaul raghavkaul temporarily deployed to gitlab July 27, 2023 22:36 — with GitHub Actions Inactive
@raghavkaul raghavkaul temporarily deployed to integration-test July 27, 2023 22:36 — with GitHub Actions Inactive
* log once
* todo

Signed-off-by: Raghav Kaul <[email protected]>
@raghavkaul raghavkaul temporarily deployed to gitlab July 28, 2023 14:50 — with GitHub Actions Inactive
@raghavkaul raghavkaul temporarily deployed to integration-test July 28, 2023 14:50 — with GitHub Actions Inactive
@raghavkaul raghavkaul enabled auto-merge (squash) July 28, 2023 17:33
@raghavkaul raghavkaul temporarily deployed to gitlab July 28, 2023 17:33 — with GitHub Actions Inactive
@raghavkaul raghavkaul temporarily deployed to integration-test July 28, 2023 17:34 — with GitHub Actions Inactive
@raghavkaul raghavkaul merged commit e9495bb into ossf:main Jul 28, 2023
ashearin pushed a commit to kgangerlm/scorecard-gitlab that referenced this pull request Nov 13, 2023
* log msg

Signed-off-by: Raghav Kaul <[email protected]>

* address pr comments

* log once
* todo

Signed-off-by: Raghav Kaul <[email protected]>

---------

Signed-off-by: Raghav Kaul <[email protected]>
Signed-off-by: Allen Shearin <[email protected]>
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.

2 participants