Skip to content

[Pipelines] Document minimum roll-interval values for R2 sinks#30883

Merged
sejoker merged 2 commits into
productionfrom
yevgen/document-pipelines-sink-min-frequency
May 18, 2026
Merged

[Pipelines] Document minimum roll-interval values for R2 sinks#30883
sejoker merged 2 commits into
productionfrom
yevgen/document-pipelines-sink-min-frequency

Conversation

@sejoker

@sejoker sejoker commented May 18, 2026

Copy link
Copy Markdown
Contributor

Summary

Screenshots (optional)

Documentation checklist

  • Is there a changelog entry (guidelines)? If you don't add one for something awesome and new (however small) — how will our customers find out? Changelogs are automatically posted to RSS feeds, the Discord, and X.
  • The change adheres to the documentation style guide.
  • If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.
  • Files which have changed name or location have been allocated redirects.

@github-actions

Copy link
Copy Markdown
Contributor

This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:

Pattern Owners
/src/content/docs/pipelines/ @Marcinthecloud, @cmackenzie1, @oliy, @garvit-gupta, @sejoker, @jonesphillip, @elithrar, @cloudflare/product-owners

@ask-bonk ask-bonk Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

1 suggestion posted.

LOW — Grammar issue in line 69. Missing article before .

The PR title is accurate and the changes are in scope. The body is missing a filled-in summary.

@ask-bonk ask-bonk Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

1 suggestion for grammar fix.

The PR title is accurate and the changes are in scope. The body is missing a filled-in summary.

Comment thread src/content/docs/pipelines/sinks/available-sinks/r2-data-catalog.mdx Outdated
@ask-bonk ask-bonk Bot added the documentation Documentation edits label May 18, 2026
@ask-bonk

ask-bonk Bot commented May 18, 2026

Copy link
Copy Markdown
Contributor

1 suggestion posted.

LOW — Grammar issue in src/content/docs/pipelines/sinks/available-sinks/r2-data-catalog.mdx line 69. Missing article before "compaction process". An inline suggestion has been added.

The PR title is accurate, the changes are in scope, and the documentation content is technically correct. The only other note is that the PR body summary is blank (template left unfilled).

Applied label: documentation.

github run

…og.mdx

Co-authored-by: ask-bonk[bot] <249159057+ask-bonk[bot]@users.noreply.github.com>
@sejoker sejoker merged commit ce3c5fb into production May 18, 2026
15 checks passed
@sejoker sejoker deleted the yevgen/document-pipelines-sink-min-frequency branch May 18, 2026 19:14
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.

6 participants