Skip to content

Link to the filtering resources guide on maven.apache.org - #561

Merged
slachiewicz merged 1 commit into
apache:masterfrom
slachiewicz:guide-filtering
Oct 31, 2026
Merged

slachiewicz merged 1 commit into
apache:masterfrom
slachiewicz:guide-filtering

Conversation

@slachiewicz

Copy link
Copy Markdown
Member

Removes the filter, escape-filtering and encoding examples, which now live in one guide on maven.apache.org. The binaries and properties-file examples stay.

Merge after apache/maven-site#1758 is published. Part of apache/maven-site#1751

@slachiewicz slachiewicz added the documentation Improvements or additions to documentation label Oct 31, 2026
@slachiewicz
slachiewicz marked this pull request as ready for review October 31, 2026 21:05

@gnodet-bot gnodet-bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Hold — prerequisite guide not yet published

The PR body says:

Merge after apache/maven-site#1758 is published

As of today, maven-site#1758 is still open, and https://maven.apache.org/guides/mini/guide-filtering-resources.html currently returns HTTP 404.

Merging now would delete ~305 lines of documentation (encoding.md.vm, escape-filtering.md.vm, filter.md) and replace the filtering topic with a dead link. Please hold until the guide is live at the target URL.

This review was generated by an AI agent, Hermès on behalf of @gnodet.


The following examples show how to use the Resources Plugin in more advanced use cases:
The [guide to filtering resources](https://maven.apache.org/guides/mini/guide-filtering-resources.html) covers variables, filter files, escaping and encoding. The following examples show how to use the Resources Plugin in more advanced use cases:

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔗 Broken link — this URL currently returns HTTP 404. It will only become valid after maven-site#1758 is merged and published.

Comment thread src/site/site.xml
<item name="Specifying a character encoding scheme"
href="/examples/encoding.html"/>
<item name="Filtering resources (guide)"
href="https://maven.apache.org/guides/mini/guide-filtering-resources.html"/>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Suggestion (not a blocker): the label Filtering resources (guide) replaces three separate items (Specifying a character encoding scheme, Filtering, Escape Filtering). Users searching the nav for encoding will no longer find it under that term. Consider Filtering & encoding (guide) to preserve discoverability.

@slachiewicz
slachiewicz merged commit 1fe3ff8 into apache:master Oct 31, 2026
22 checks passed
@github-actions github-actions Bot added this to the 3.5.1 milestone Oct 31, 2026
@slachiewicz
slachiewicz deleted the guide-filtering branch October 31, 2026 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants