Skip to content

Site: Remove duplicate 'for' in markdown_testing.py print output#4514

Merged
dimas-b merged 1 commit into
apache:mainfrom
visit2rahul:fix/duplicate-for-typo
May 21, 2026
Merged

Site: Remove duplicate 'for' in markdown_testing.py print output#4514
dimas-b merged 1 commit into
apache:mainfrom
visit2rahul:fix/duplicate-for-typo

Conversation

@visit2rahul

Copy link
Copy Markdown
Contributor

Trivial typo fix in a CI tooling script. The print statement at site/it/site_checks/markdown_testing.py:122 reads "Test steps durations for for {md_file}" - duplicate "for". Removed one to make it "Test steps durations for {md_file}". No logic change; output-only.

Originated in #3553 (CI for guides).

@github-project-automation github-project-automation Bot moved this from PRs In Progress to Ready to merge in Basic Kanban Board May 21, 2026
@visit2rahul

visit2rahul commented May 21, 2026

Copy link
Copy Markdown
Contributor Author

Thank you @snazy for the quick review.

The CI failure is in InMemoryBufferEventListenerBufferSizeTest > testProcessorFailureRecovery() - a concurrency/recovery test in the Java event-listener subsystem. This PR only changes a single Python print statement in site/it/site_checks/markdown_testing.py, so the failure is unrelated. Reads like a flaky timing-sensitive test.

@snazy could you re-run the failed job when convenient? A re-run should clear it. Happy to file a follow-up issue for the underlying flake if helpful.

Please review as your time permits.

@visit2rahul visit2rahul requested a review from snazy May 21, 2026 11:10
@dimas-b dimas-b enabled auto-merge (squash) May 21, 2026 15:03
@dimas-b dimas-b merged commit 23aa800 into apache:main May 21, 2026
43 of 45 checks passed
@github-project-automation github-project-automation Bot moved this from Ready to merge to Done in Basic Kanban Board May 21, 2026
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.

3 participants