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

Update model.md: fix typo in URL #290

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

evgkrsk
Copy link

@evgkrsk evgkrsk commented Jan 13, 2023

No description provided.

@@ -16,7 +16,7 @@ Components are loaded from jsonnet, json, or yaml files. Only jsonnet files prov
to customize object definitions for different environments. In addition, qbec support loading
objects from helm charts, and using jsonnet libraries to produce them.

More details can be found in the [authoring components](userguide/authoring/) page.
More details can be found in the [authoring components](usage/authoring/) page.
Copy link
Contributor

Choose a reason for hiding this comment

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

usage is anchored at the same level as model. Needs a relative reference like ../usage/authoring

Copy link
Contributor

Choose a reason for hiding this comment

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

Btw, thanks for raising this

Copy link
Author

Choose a reason for hiding this comment

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

Okay, done.

@github-actions
Copy link

github-actions bot commented Mar 5, 2023

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@evgkrsk
Copy link
Author

evgkrsk commented Mar 6, 2023

I have read the CLA Document and I hereby sign the CLA

@taitken-splunk
Copy link
Contributor

recheck

rdoshi1 added a commit to splunk/cla-agreement that referenced this pull request Oct 23, 2023
@taitken-splunk
Copy link
Contributor

Sorry for the long delay, I am now taking up maintaining this project, so if you could rebase this PR we can get it merged!

@evgkrsk
Copy link
Author

evgkrsk commented Oct 24, 2023

I have read the Code of Conduct and I hereby accept the Terms

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.

3 participants