Skip to content

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

Open
apify-factory wants to merge 2 commits into
masterfrom
apify-factory/factory-review
Open

apify-factory wants to merge 2 commits into
masterfrom
apify-factory/factory-review

Conversation

@apify-factory

Copy link
Copy Markdown

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/_charging.py Pay-per-event ChargingManager that computes and submits charges to users against pricing info; any change can alter what customers are billed. 1
src/apify/_crypto.py RSA/AES encryption, private-key loading, secret-input decryption and HMAC signing; subtle changes can leak secrets or break cryptographic guarantees. 1
.rules.md Instruction file for coding agents (duplicate of CLAUDE.md) that steers automated contributors; changes to agent guidance need human review. 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 93.05%. Comparing base (3ed1a48) to head (04bb06d).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1182   +/-   ##
=======================================
  Coverage   93.05%   93.05%           
=======================================
  Files          54       54           
  Lines        3671     3671           
=======================================
  Hits         3416     3416           
  Misses        255      255           
Flag Coverage Δ
e2e 31.73% <ø> (ø)
integration 51.70% <ø> (ø)
unit 89.78% <ø> (ø)

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.

This branch has not been deployed

No deployments
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.

1 participant