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

Slight update to README (as per installation testing) #33

Merged
merged 1 commit into from
Feb 26, 2024

Conversation

tpmccallum
Copy link
Contributor

Signed-off-by: tpmccallum tim.mccallum@fermyon.com

Signed-off-by: tpmccallum tim.mccallum@fermyon.com
@@ -43,7 +50,7 @@ make install
## Prerequisites

Ensure spin-operator is installed in your Kubernetes cluster. See the [spin-operator Quickstart
Guide](https://github.com/spinkube/spin-operator/blob/main/documentation/content/quickstart.md).
Guide](https://github.com/spinkube/documentation/blob/main/content/en/docs/spin-operator/quickstart/_index.md).
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Point to the new single source of truth for documentation (spinkube/documentation repository)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can update this Quickstart URL and some other repeatable URLs e.g. Prerequisites to live URLs once the documentation is published and public facing.

Fetch the plugin:

```sh
git clone git@github.com:spinkube/spin-plugin-k8s.git
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Listing the fetch as to mirror all of the steps required to perform the installation from source.

@tpmccallum tpmccallum merged commit 5ea8c60 into main Feb 26, 2024
1 check 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.

None yet

2 participants