Skip to content

ci: unify auto-update PR title and description format - #1213

Merged
vdusek merged 1 commit into
masterfrom
ci/unify-update-pr-format
Nov 9, 2026
Merged

vdusek merged 1 commit into
masterfrom
ci/unify-update-pr-format

Conversation

@vdusek

@vdusek vdusek commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

Brings the PRs opened by the Regenerate models workflow to the same format as the matching workflows in apify-client-js and crawlee-python.

  • Titles follow chore: <Verb> <object>.
  • Descriptions are short bullet lists wrapped at 72 columns, so the squash commit body reads well in git log. No blockquotes.
  • A footer names the workflow and links its run.

✍️ Drafted by Claude Code

@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 Nov 9, 2026
@vdusek vdusek self-assigned this Nov 9, 2026
@vdusek
vdusek requested a review from apify-factory November 9, 2026 06:47

@apify-factory apify-factory 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.

Looks good, nothing blocking.

🏭 Factory review — ℹ️ not auto-approved

This change was reviewed but cannot be auto-approved: a gate failed, so the review cannot end in an approval.

check result
Static gates ❌ 14/16 — file-extensions, deny-globs
Review findings ✅ none
Reviewer escalation ✅ none
Juror 1 — claude-sonnet-5-5, adversarial ⏭️ skipped — a gate failed
Why it was not auto-approved

Failed gates:

  • file-extensions: disallowed extension: .github/workflows/on_schedule_regenerate_models.yaml
  • deny-globs: .github/workflows/on_schedule_regenerate_models.yaml matches .github/**

The comments above are the review itself; this section only explains the approval decision. Push a change and request @apify-factory again to re-run it, or ask a human to review.

Reviewed d0db9567b · workflow run

@codecov

codecov Bot commented Nov 9, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.73%. Comparing base (503d9d9) to head (d0db956).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1213      +/-   ##
==========================================
- Coverage   95.74%   95.73%   -0.02%     
==========================================
  Files          63       63              
  Lines        6676     6676              
==========================================
- Hits         6392     6391       -1     
- Misses        284      285       +1     
Flag Coverage Δ
integration 91.17% <ø> (-0.06%) ⬇️
unit 90.68% <ø> (-0.02%) ⬇️

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 requested a review from Mantisus November 9, 2026 06:49

@apify-approval-mirror apify-approval-mirror Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Mirroring approval from @Mantisus.

@vdusek
vdusek merged commit e521a24 into master Nov 9, 2026
39 of 43 checks passed
@vdusek
vdusek deleted the ci/unify-update-pr-format branch November 9, 2026 11:37
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