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

ko: update repo location #121332

Conversation

developer-guy
Copy link
Contributor

@developer-guy developer-guy commented Jan 23, 2023

Signed-off-by: Batuhan Apaydın [email protected]

  • Have you followed the guidelines for contributing?
  • Have you ensured that your commits follow the commit style guide?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

ko has been moved out to its new repository named ko-build1, this PR aims to replace the ko's repository URL with the most recent one.

/cc @mattmoor @imjasonh

Footnotes

  1. https://github.com/ko-build/ko/issues/791

@BrewTestBot BrewTestBot added the go Go use is a significant feature of the PR or issue label Jan 23, 2023
@SMillerDev
Copy link
Member

Did the ldflag to set the version also change?

@chenrui333 chenrui333 added the CI-no-bottles Merge without publishing bottles label Jan 23, 2023
@chenrui333 chenrui333 changed the title feature: s/google/ko-build/g ko: update repo location Jan 23, 2023
@developer-guy
Copy link
Contributor Author

AFAIK, no need to update the ldflag, PTAL @imjasonh

Formula/ko.rb Outdated Show resolved Hide resolved
Formula/ko.rb Outdated Show resolved Hide resolved
@developer-guy developer-guy force-pushed the feature/fix-ko-repository-url branch 2 times, most recently from 8070b52 to a472b45 Compare January 23, 2023 16:20
@chenrui333
Copy link
Member

@developer-guy let's follow the commit style in here, https://docs.brew.sh/Formula-Cookbook#commit

@chenrui333 chenrui333 added CI-syntax-only Change only affects brew syntax, not the install. Only run syntax CI. and removed CI-no-bottles Merge without publishing bottles labels Jan 23, 2023
Signed-off-by: Batuhan Apaydın <[email protected]>
@developer-guy developer-guy force-pushed the feature/fix-ko-repository-url branch from a472b45 to 428d4cf Compare January 23, 2023 16:27
@developer-guy
Copy link
Contributor Author

it should be okay right now, thanks @chenrui333

@BrewTestBot
Copy link
Member

🤖 A scheduled task has triggered a merge.

@github-actions github-actions bot added the outdated PR was locked due to age label Mar 7, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CI-syntax-only Change only affects brew syntax, not the install. Only run syntax CI. go Go use is a significant feature of the PR or issue outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants