Skip to content

Document filters as processors #1605

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 9 commits into from
Apr 24, 2025
Merged

Document filters as processors #1605

merged 9 commits into from
Apr 24, 2025

Conversation

n-my
Copy link
Contributor

@n-my n-my commented Apr 23, 2025

Fixes #1604

@n-my n-my requested review from a team as code owners April 23, 2025 19:04
alexakreizinger
alexakreizinger reviewed Apr 23, 2025
View reviewed changes
Copy link
Member

@alexakreizinger alexakreizinger left a comment

Choose a reason for hiding this comment

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

thanks for submitting a PR @n-my!

I made a few style suggestions, but otherwise LGTM :)

alexakreizinger
alexakreizinger reviewed Apr 23, 2025
View reviewed changes
alexakreizinger
alexakreizinger reviewed Apr 23, 2025
View reviewed changes
n-my and others added 9 commits April 23, 2025 20:22
@n-my
filters as processors
41e7549 
Signed-off-by: Nico Monchy <[email protected]>
@n-my
filters as processors
0e3c660 
Signed-off-by: Nico Monchy <[email protected]>
@n-my
symmary: add processors filters page
769e0b6 
Signed-off-by: Nico Monchy <[email protected]>
@n-my
pipeline: processors: filters: fix link relative path
dea157d 
Signed-off-by: Nico Monchy <[email protected]>
@n-my @alexakreizinger
Update pipeline/processors/filters.md
ea90589 
Co-authored-by: Alexa Kreizinger <[email protected]>
Signed-off-by: Nico Monchy <[email protected]>
@n-my @alexakreizinger
Update pipeline/processors/filters.md
9278e65 
Co-authored-by: Alexa Kreizinger <[email protected]>
Signed-off-by: Nico Monchy <[email protected]>
@n-my @alexakreizinger
Update pipeline/processors/filters.md
2ae90fe 
Co-authored-by: Alexa Kreizinger <[email protected]>
Signed-off-by: Nico Monchy <[email protected]>
@n-my @alexakreizinger
Update SUMMARY.md
58b9866 
Co-authored-by: Alexa Kreizinger <[email protected]>
Signed-off-by: Nico Monchy <[email protected]>
@n-my
pipeline: processors: filters: fix link to grep page
01201e6 
Signed-off-by: Nico Monchy <[email protected]>
@n-my
Copy link
Contributor Author

n-my commented Apr 23, 2025 •
edited
Loading

Thanks for the review @alexakreizinger
I committed the suggestions, and fixed the link to the grep page.

@esmerel esmerel merged commit 318b773 into fluent:master Apr 24, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alexakreizinger alexakreizinger alexakreizinger approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Document that any filter can be used as a processor
3 participants
@n-my @alexakreizinger @esmerel