Skip to content

ci: add the apify-factory review workflow - #1130

Merged
mtrunkat merged 2 commits into
masterfrom
apify-factory/factory-review
Oct 15, 2026
Merged

mtrunkat merged 2 commits into
masterfrom
apify-factory/factory-review

Conversation

@apify-factory

Copy link
Copy Markdown
Contributor

Lets apify-factory review pull requests here and approve the trivial ones.

  • Requesting apify-factory as a reviewer runs an AI review; when every gate passes, the review finds nothing and a second model agrees, it approves instead of commenting.
  • A push that changes the code, removing the request, or going back to draft withdraws the bot's own approval.
  • This repository's factory_auto_approve property is true, which is what allows approvals, and mtrunkat becomes the code owner of the workflow file.
  • These paths are never auto-approved here, on top of the defaults every repository has. An agent proposed them from the repository's layout; edit the list in the workflow like any other code.
Path Why Files
src/apify_client/_models.py Pydantic models auto-generated by datamodel-codegen from the Apify OpenAPI spec; must change only via the generator, never by hand. 1
src/apify_client/_typeddicts.py TypedDicts auto-generated from the Apify OpenAPI spec and postprocessed; must change only via the generator, never by hand. 1
src/apify_client/_literals.py Literal types auto-generated by the model postprocessing script from the OpenAPI spec; must change only via the generator, never by hand. 1

Refs https://github.com/apify/ai-developer/blob/main/actions/auto-approve/README.md

@github-actions github-actions Bot added the t-tooling Issues with this label are in the ownership of the tooling team. label Oct 14, 2026
@codecov

codecov Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.46%. Comparing base (ef8f088) to head (337db3d).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1130   +/-   ##
=======================================
  Coverage   95.46%   95.46%           
=======================================
  Files          61       61           
  Lines        5935     5935           
=======================================
  Hits         5666     5666           
  Misses        269      269           
Flag Coverage Δ
integration 90.73% <ø> (+0.13%) ⬆️
unit 88.87% <ø> (ø)

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.

@mtrunkat
mtrunkat merged commit 53a9954 into master Oct 15, 2026
36 checks passed
@mtrunkat
mtrunkat deleted the apify-factory/factory-review branch October 15, 2026 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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