Skip to content

[docs] Miscellaneous docs clean up #43370

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

Merged
merged 4 commits into from
Mar 25, 2025
Merged

Conversation

colleenmcginnis
Copy link
Contributor

@colleenmcginnis colleenmcginnis commented Mar 20, 2025

Miscellaneous docs clean up including:

@colleenmcginnis colleenmcginnis added docs Team:Docs Label for the Observability docs team labels Mar 20, 2025
@colleenmcginnis colleenmcginnis self-assigned this Mar 20, 2025
@botelastic botelastic bot added needs_team Indicates that the issue/PR needs a Team:* label and removed needs_team Indicates that the issue/PR needs a Team:* label labels Mar 20, 2025
Copy link
Contributor

mergify bot commented Mar 20, 2025

This pull request does not have a backport label.
If this is a bug or security fix, could you label this PR @colleenmcginnis? 🙏.
For such, you'll need to label your PR with:

  • The upcoming major version of the Elastic Stack
  • The upcoming minor version of the Elastic Stack (if you're not pushing a breaking change)

To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-8./d is the label to automatically backport to the 8./d branch. /d is the digit
  • backport-active-all is the label that automatically backports to all active branches.
  • backport-active-8 is the label that automatically backports to all active minor branches for the 8 major.
  • backport-active-9 is the label that automatically backports to all active minor branches for the 9 major.

@colleenmcginnis colleenmcginnis marked this pull request as ready for review March 21, 2025 14:26
@colleenmcginnis colleenmcginnis requested a review from a team as a code owner March 21, 2025 14:26
@colleenmcginnis colleenmcginnis added the backport-skip Skip notification from the automated backport with mergify label Mar 21, 2025
@colleenmcginnis
Copy link
Contributor Author

@kilfoyle we didn't backport to 9.0, right?

@colleenmcginnis colleenmcginnis enabled auto-merge (squash) March 21, 2025 14:27
@kilfoyle
Copy link
Contributor

kilfoyle commented Mar 21, 2025

@colleenmcginnis No, backporting hadn't occurred to me so #42897 is only in main.

I guess it needs to be backported but I'm a little nervous about that given the complexity. Shall we try?

I'm actually not sure if the plan is to serve the docs out of main or 9.0.

LGTM for the changes. Thanks!

@colleenmcginnis
Copy link
Contributor Author

@rdner can you please take a look?

@kilfoyle
Copy link
Contributor

@leehinman Would you maybe be able to approve and merge this? I think it's a bit urgent.

@leehinman leehinman removed the request for review from rdner March 25, 2025 18:03
@leehinman
Copy link
Contributor

@leehinman Would you maybe be able to approve and merge this? I think it's a bit urgent.

approved, but I don't see a merge "button".

@colleenmcginnis
Copy link
Contributor Author

I disabled auto-merge and the button came back, but I get this when I try to click it:

image

@karenzone
Copy link
Contributor

I tried and got the same message, @colleenmcginnis

@jlind23 jlind23 merged commit 9b76301 into elastic:main Mar 25, 2025
11 checks passed
@colleenmcginnis colleenmcginnis added backport-9.0 Automated backport to the 9.0 branch and removed backport-skip Skip notification from the automated backport with mergify labels Mar 26, 2025
@colleenmcginnis colleenmcginnis deleted the docs-clean-up branch March 26, 2025 00:44
mergify bot pushed a commit that referenced this pull request Mar 26, 2025
* remove unused substitutions

* update redirected links

(cherry picked from commit 9b76301)
colleenmcginnis added a commit that referenced this pull request Apr 17, 2025
* remove unused substitutions

* update redirected links

(cherry picked from commit 9b76301)

Co-authored-by: Colleen McGinnis <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-9.0 Automated backport to the 9.0 branch docs Team:Docs Label for the Observability docs team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants