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

✨ Consider ko-build/setup-ko as a packaging workflow #2692

Merged
merged 1 commit into from
Feb 23, 2023

Conversation

imjasonh
Copy link
Contributor

What kind of change does this PR introduce?

Slight feature / bug fix. The imjasonh/setup-ko workflow was moved to https://github.com/ko-build, and thanks to GitHub redirect magic, both names can be used in a workflow -- the imjasonh ref redirects to the ko-build one.

What is the current behavior?

Workflows using the old imjasonh/setup-ko will not be matched as using a packaging workflow.

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

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

Which issue(s) this PR fixes

NONE

Special notes for your reviewer

NONE

Does this PR introduce a user-facing change?

ko-build/setup-ko is detected as a packaging workflow

Signed-off-by: Jason Hall <jason@chainguard.dev>
@naveensrinivasan
Copy link
Member

Thank you, @imjasonh!

@naveensrinivasan naveensrinivasan enabled auto-merge (squash) February 23, 2023 16:57
@naveensrinivasan naveensrinivasan merged commit 78069d8 into ossf:main Feb 23, 2023
Shofiya2003 pushed a commit to Shofiya2003/scorecard that referenced this pull request Mar 10, 2023
Signed-off-by: Jason Hall <jason@chainguard.dev>
Signed-off-by: Shofiya2003 <shofiyabootwala@gmail.com>
Shofiya2003 pushed a commit to Shofiya2003/scorecard that referenced this pull request Mar 10, 2023
Signed-off-by: Jason Hall <jason@chainguard.dev>
Signed-off-by: Shofiya2003 <shofiyabootwala@gmail.com>
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