Skip to content

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

Merged
apify-service-account merged 3 commits into
masterfrom
apify-factory/factory-review
Oct 19, 2026
Merged

apify-service-account merged 3 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/_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.07%. Comparing base (3ed1a48) to head (4687448).
⚠️ Report is 6 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1182      +/-   ##
==========================================
+ Coverage   93.05%   93.07%   +0.01%     
==========================================
  Files          54       54              
  Lines        3671     3680       +9     
==========================================
+ Hits         3416     3425       +9     
  Misses        255      255              
Flag Coverage Δ
e2e 31.65% <ø> (-0.08%) ⬇️
integration 51.73% <ø> (+0.03%) ⬆️
unit 89.83% <ø> (+0.05%) ⬆️

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.

@mfori
mfori removed the request for review from mtrunkat October 19, 2026 14:41
@apify-service-account
apify-service-account merged commit c27098a into master Oct 19, 2026
31 checks passed
@apify-service-account
apify-service-account deleted the apify-factory/factory-review branch October 19, 2026 15:04
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