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

docs: fixes broken link to CLI docs on the readme #1575

Merged
merged 1 commit into from
Nov 7, 2023

Conversation

andresgalante
Copy link
Contributor

There is a broken link on the read me. This PR is fixing it :)

@CLAassistant
Copy link

CLAassistant commented Nov 7, 2023

CLA assistant check
All committers have signed the CLA.

@JAORMX
Copy link
Contributor

JAORMX commented Nov 7, 2023

Thanks a lot for the contribution! Once CI is happy and the CLA is signed, we'll get this merged.

@andresgalante
Copy link
Contributor Author

andresgalante commented Nov 7, 2023

Actually, should it be linked to here instead: https://minder-docs.stacklok.dev/ref/api ? nevermind, I got the URL right the first time :)

@JAORMX
Copy link
Contributor

JAORMX commented Nov 7, 2023

Actually, should it be linked to here instead: https://minder-docs.stacklok.dev/ref/api ?

I think the ref you linked it to is just fine. Though it would be handy to also have a reference to the API handy.

@JAORMX JAORMX merged commit 282146e into mindersec:main Nov 7, 2023
11 checks passed
@andresgalante
Copy link
Contributor Author

I agree, since I have it open let me add a reference to the API too

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.

4 participants