Skip to content

Conversation

mashhurs
Copy link
Contributor

Checked on my local if missed the plugin docs between 2025-07-15 ~ 2025-07-28 and got this commit.

@mashhurs
Copy link
Contributor Author

😨 I have no idea what INFO:build_docs:asciidoctor: WARNING: invalid reference: hash error mean!

[cols="<,<,<",options="header",]
|=======================================================================
|Setting |Input type|Required
| <<{version}-plugins-{type}s-{plugin}-aggregation_fields>> |<<hash,hash>>|No
Copy link
Contributor

Choose a reason for hiding this comment

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

#1910 (comment)

This manually generated PR is using the old <<hash,hash>> (8x and Asciidoc compatible), but the references should be in this format: {logstash-ref}/configuration-file-structure.html#hash[hash] to link correctly in 9.0+.

Off the top of my head, I'm not sure why this isn't a problem for auto-generated plugin docs. (Check out the PR where 4.2.0 was updated.)

Screenshot 2025-09-15 at 7 54 39 PM

Copy link
Contributor

@karenzone karenzone Sep 16, 2025

Choose a reason for hiding this comment

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

To be clear, it's not just the format that has changed. The file location has changed, too.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah didn't notice that we had --repair option, will fix this in a moment.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Docs got updated and {logstash-ref}/configuration-file-structure.html#hash[hash] changes appeared, was missed due to --repair option on my local.
CI seems also 🟢 happy!

@karenzone
Copy link
Contributor

😨 I have no idea what INFO:build_docs:asciidoctor: WARNING: invalid reference: hash error mean!

It's a link format thing, and ci would error on every single one (hash, field, boolean, array, etc.) I left a note inline to illustrate.

Copy link
Contributor

@karenzone karenzone left a comment

Choose a reason for hiding this comment

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

LGTM!

@mashhurs mashhurs merged commit 9cc46f6 into elastic:versioned_plugin_docs Sep 18, 2025
2 checks passed
@mashhurs mashhurs deleted the missed-plugin-docs branch September 18, 2025 21:49
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.

Check if plugin docs missed between 2025-07-15 ~ 2025-07-28
2 participants