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

Patch 3 #201

Merged
merged 2 commits into from
Oct 4, 2024
Merged

Patch 3 #201

merged 2 commits into from
Oct 4, 2024

Conversation

joergi
Copy link
Contributor

@joergi joergi commented Sep 27, 2024

As I was struggling with the installation (as a non GO user), I added this to the Readme, so it will maybe help others

  • adds the frizbee path to your PATH

  • go get -u github.com/stacklok/frizbee
    'go get' is no longer supported outside a module.

  • go: 'go install' requires a version when current directory is not in a module

adds the frizbee path to your PATH
`go get -u github.com/stacklok/frizbee` 
'go get' is no longer supported outside a module.

go: 'go install' requires a version when current directory is not in a module
@CLAassistant
Copy link

CLAassistant commented Sep 27, 2024

CLA assistant check
All committers have signed the CLA.

@coveralls
Copy link

Coverage Status

coverage: 52.747%. remained the same
when pulling b9f64a3 on joergi:patch-3
into 90dcc82 on stacklok:main.

@JAORMX
Copy link
Contributor

JAORMX commented Oct 1, 2024

@joergi your PR looks good to me! can you sign the CLI so we can move forward?

@staceypotter
Copy link
Contributor

CLA assistant check Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@joergi - are you able to sign the CLA to get this change incorporated? Thanks!

@joergi
Copy link
Contributor Author

joergi commented Oct 4, 2024

It should be signed...

@puerco puerco merged commit 9d170e7 into stacklok:main Oct 4, 2024
10 checks passed
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.

6 participants