Skip to content

ci: call the apify-factory review through apify/workflows - #1154

Merged
apify-service-account merged 1 commit into
masterfrom
mfori/factory-review-public-workflow
Oct 19, 2026
Merged

apify-service-account merged 1 commit into
masterfrom
mfori/factory-review-public-workflow

Conversation

@mfori

@mfori mfori commented Oct 19, 2026

Copy link
Copy Markdown
Member

Points the apify-factory review at the public reusable workflow in apify/workflows.

  • Every review run here failed before starting: a public repository cannot call a workflow in the internal apify/ai-developer, which the review used to call directly.
  • The public workflow runs the same pipeline, so reviews, approvals and this repository's deny globs stay as they were; it already approved a PR end to end in apify-sdk-js.

Refs apify/apify-sdk-js#817


Generated by Claude Code

@mfori
mfori requested a review from mtrunkat as a code owner October 19, 2026 14:39
@github-actions github-actions Bot added the t-tooling Issues with this label are in the ownership of the tooling team. label Oct 19, 2026
@mfori mfori added adhoc Ad-hoc unplanned task added during the sprint. and removed t-tooling Issues with this label are in the ownership of the tooling team. labels Oct 19, 2026 — with Claude
@apify-service-account apify-service-account added the t-console Issues with this label are in the ownership of the console team. label Oct 19, 2026
@mfori
mfori removed the request for review from mtrunkat October 19, 2026 14:40
@codecov

codecov Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

Codecov Report

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

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1154   +/-   ##
=======================================
  Coverage   95.53%   95.53%           
=======================================
  Files          61       61           
  Lines        5974     5974           
=======================================
  Hits         5707     5707           
  Misses        267      267           
Flag Coverage Δ
integration 90.75% <ø> (-0.16%) ⬇️
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.

@apify-service-account
apify-service-account merged commit ed76532 into master Oct 19, 2026
35 of 39 checks passed
@apify-service-account
apify-service-account deleted the mfori/factory-review-public-workflow branch October 19, 2026 14: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-console Issues with this label are in the ownership of the console team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants