Skip to content

ci: use a TODO placeholder title for model-regeneration PRs#901

Merged
vdusek merged 3 commits into
masterfrom
ci/regenerate-models-placeholder-title
Jun 26, 2026
Merged

ci: use a TODO placeholder title for model-regeneration PRs#901
vdusek merged 3 commits into
masterfrom
ci/regenerate-models-placeholder-title

Conversation

@vdusek

@vdusek vdusek commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

The auto-opened model-regeneration PR used to default its title to [TODO]: update generated models from apify-docs PR #N. That looked finished enough to merge as-is and tied the title to apify-docs, even though the right title depends on the actual model diff (a routine chore: sync, or a refactor:/feat:/fix: when models meaningfully change).

The PR title is now a bare TODO placeholder. It carries no valid Conventional Commits type, so pr-title-check stays red until a human replaces it. The apify-docs reference and link, plus a > [!IMPORTANT] note asking the assignee to replace both the title and the description, now live in the PR body. The automated regeneration commit keeps its own descriptive chore: message (separate from the PR title).

@vdusek vdusek 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 Jun 25, 2026
@vdusek vdusek self-assigned this Jun 25, 2026
@codecov

codecov Bot commented Jun 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.54%. Comparing base (8c48a06) to head (2256199).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #901      +/-   ##
==========================================
+ Coverage   94.52%   94.54%   +0.01%     
==========================================
  Files          48       48              
  Lines        5098     5098              
==========================================
+ Hits         4819     4820       +1     
+ Misses        279      278       -1     
Flag Coverage Δ
integration 92.93% <ø> (+0.31%) ⬆️
unit 83.36% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@vdusek vdusek marked this pull request as ready for review June 26, 2026 09:09
@vdusek vdusek requested a review from Pijukatel June 26, 2026 09:10
@vdusek vdusek merged commit d48202b into master Jun 26, 2026
30 checks passed
@vdusek vdusek deleted the ci/regenerate-models-placeholder-title branch June 26, 2026 09:54
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.

3 participants