Skip to content

feat: Add a counter of in-flight rendering type detections#3355

Merged
janbuchar merged 1 commit intomasterfrom
add-inflight-detection-counter
Jan 20, 2026
Merged

feat: Add a counter of in-flight rendering type detections#3355
janbuchar merged 1 commit intomasterfrom
add-inflight-detection-counter

Conversation

@janbuchar
Copy link
Copy Markdown
Contributor

@janbuchar janbuchar commented Jan 20, 2026

  • the added property is useful for scrapers such as Website Content Crawler - it allows waiting for detections to be finished before shutting down the process

@janbuchar janbuchar added adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team. labels Jan 20, 2026
@janbuchar janbuchar requested review from B4nan and barjin January 20, 2026 12:00
@github-actions github-actions Bot added this to the 132nd sprint - Tooling team milestone Jan 20, 2026
Copy link
Copy Markdown
Member

@barjin barjin left a comment

Choose a reason for hiding this comment

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

lgtm code-wise 👍

also, what is this for?

@janbuchar
Copy link
Copy Markdown
Contributor Author

also, what is this for?

Right, sorry. Updated the PR description.

Copy link
Copy Markdown
Member

@B4nan B4nan left a comment

Choose a reason for hiding this comment

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

the _runRequestHandler is getting pretty huge, likely worth splitting to more pieces, but we dont need to deal with that here

@janbuchar
Copy link
Copy Markdown
Contributor Author

the _runRequestHandler is getting pretty huge, likely worth splitting to more pieces, but we dont need to deal with that here

Oh yeah. It'll be a bit better in v4 I guess.

@janbuchar janbuchar merged commit 565fc34 into master Jan 20, 2026
10 checks passed
@janbuchar janbuchar deleted the add-inflight-detection-counter branch January 20, 2026 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants