Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

✨ Switched API release links to non-API release links #4290

Merged
merged 1 commit into from
Aug 8, 2024

Conversation

klbynum
Copy link
Contributor

@klbynum klbynum commented Aug 7, 2024

What kind of change does this PR introduce?

This PR introduces a feature change for the release links generated by the following command:
go run main.go --repo ossf/scorecard --checks Signed-Releases --format json --show-details | jq

What is the current behavior?

When running command to view releases, links display API format

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

When running command to view releases, links display non-API format

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

Which issue(s) this PR fixes

Fixes #4030

Special notes for your reviewer

Did not add test for changes but ran current test

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

NONE

@klbynum klbynum requested a review from a team as a code owner August 7, 2024 18:35
@klbynum klbynum requested review from naveensrinivasan and justaugustus and removed request for a team August 7, 2024 18:35
@spencerschrock
Copy link
Member

/scdiff generate Signed-Releases

Copy link

github-actions bot commented Aug 7, 2024

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! Can you please fix the DCO status? We cannot merge without it.

You can find remediation steps here: https://github.com/ossf/scorecard/pull/4290/checks?check_run_id=28478870940

Co-authored-by: Silas Morgan <victorsmorgan@gmail.com>
Co-authored-by: Jordin Jones <j.a.jones107732@spartans.nsu.edu>
Signed-off-by: Kemon Bynum <monbynum51@icloud.com>
@klbynum
Copy link
Contributor Author

klbynum commented Aug 8, 2024

Hey @spencerschrock, thank you for taking a look at this PR. We followed the remediation steps and passed the DCO check.

@klbynum klbynum temporarily deployed to integration-test August 8, 2024 18:11 — with GitHub Actions Inactive
@spencerschrock spencerschrock merged commit 642c844 into ossf:main Aug 8, 2024
36 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Include human friendly links for the GitHub artifacts for Signed-Releases
2 participants