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

⚠️ Remove the old Details field from CheckResult #1906

Merged
merged 1 commit into from May 12, 2022

Conversation

naveensrinivasan
Copy link
Member

 #1393

Signed-off-by: naveensrinivasan <172697+naveensrinivasan@users.noreply.github.com>
@naveensrinivasan naveensrinivasan changed the title :warn: Remove the old Details field from CheckResult ⚠️ Remove the old Details field from CheckResult May 12, 2022
@naveensrinivasan naveensrinivasan temporarily deployed to integration-test May 12, 2022 19:26 Inactive
@naveensrinivasan naveensrinivasan enabled auto-merge (squash) May 12, 2022 19:28
@codecov
Copy link

codecov bot commented May 12, 2022

Codecov Report

Merging #1906 (d3752ed) into main (b9f333b) will increase coverage by 3.02%.
The diff coverage is 57.14%.

@@            Coverage Diff             @@
##             main    #1906      +/-   ##
==========================================
+ Coverage   51.52%   54.54%   +3.02%     
==========================================
  Files          79       79              
  Lines        6787     6787              
==========================================
+ Hits         3497     3702     +205     
+ Misses       3059     2848     -211     
- Partials      231      237       +6     

@github-actions
Copy link

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

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 merged commit 0275a94 into main May 12, 2022
@naveensrinivasan naveensrinivasan deleted the naveen/feat/remove-details-checkresult branch May 12, 2022 19:58
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.

None yet

2 participants