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] Clean up cross-repo links #2640

Merged
merged 2 commits into from
Mar 3, 2025

Conversation

colleenmcginnis
Copy link
Contributor

Follow up to #2635

When using cross-repo links, the path should be relative to the docset.yml not the full path within the repo (updated docs-builder docs).

@colleenmcginnis colleenmcginnis enabled auto-merge (squash) February 28, 2025 20:09
@colleenmcginnis colleenmcginnis requested a review from JoshMock March 3, 2025 16:02
@colleenmcginnis
Copy link
Contributor Author

@JoshMock can you help with these pending checks?

@JoshMock
Copy link
Member

JoshMock commented Mar 3, 2025

we can skip the pending checks, they assume all PRs will include code changes, which is almost always true.

@JoshMock JoshMock disabled auto-merge March 3, 2025 21:28
@JoshMock JoshMock merged commit 8ca68a4 into elastic:main Mar 3, 2025
10 checks passed
@colleenmcginnis colleenmcginnis deleted the docs-fix-cross-repo-links branch March 3, 2025 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants