Skip to content

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

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:49
@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
@mfori
mfori removed the request for review from mtrunkat October 19, 2026 14:50
@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 94.15%. Comparing base (508f544) to head (09162af).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2341      +/-   ##
==========================================
+ Coverage   94.11%   94.15%   +0.03%     
==========================================
  Files         184      184              
  Lines       13644    13644              
==========================================
+ Hits        12841    12846       +5     
+ Misses        803      798       -5     
Flag Coverage Δ
unit 94.15% <ø> (+0.03%) ⬆️

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 added the t-console Issues with this label are in the ownership of the console team. label Oct 19, 2026
@apify-service-account
apify-service-account merged commit 610fbcb into master Oct 19, 2026
43 of 44 checks passed
@apify-service-account
apify-service-account deleted the mfori/factory-review-public-workflow branch October 19, 2026 15:01
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