Skip to content

fix(ci): keep nightly e2e alert outside reusable workflow#3312

Merged
serrrfirat merged 1 commit into
nearai:mainfrom
serrrfirat:fix/main-run-tests-startup
May 6, 2026
Merged

fix(ci): keep nightly e2e alert outside reusable workflow#3312
serrrfirat merged 1 commit into
nearai:mainfrom
serrrfirat:fix/main-run-tests-startup

Conversation

@serrrfirat
Copy link
Copy Markdown
Collaborator

Summary

  • remove the scheduled alert job from reusable .github/workflows/e2e.yml
  • add a dedicated Nightly E2E wrapper workflow that schedules full E2E and reports issues on scheduled failures

Why

Run Tests on main has been failing at workflow startup since #3293. The reusable e2e.yml gained a schedule-only alert job requesting issues: write. Even though the e2e jobs in test.yml are conditional, GitHub validates the referenced reusable workflow and fails startup before creating jobs.

Verification

  • Parsed .github/workflows/e2e.yml, .github/workflows/nightly-e2e.yml, and .github/workflows/test.yml with Ruby YAML parser.

Expected effect: the next main push should start Run Tests normally instead of startup_failure.

@gemini-code-assist
Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@github-actions github-actions Bot added scope: ci CI/CD workflows size: M 50-199 changed lines risk: medium Business logic, config, or moderate-risk modules contributor: core 20+ merged PRs labels May 6, 2026
@serrrfirat serrrfirat force-pushed the fix/main-run-tests-startup branch from 20540cf to 4c30395 Compare May 6, 2026 20:55
@serrrfirat serrrfirat merged commit 89c21aa into nearai:main May 6, 2026
29 checks passed
@serrrfirat serrrfirat deleted the fix/main-run-tests-startup branch May 6, 2026 21:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contributor: core 20+ merged PRs risk: medium Business logic, config, or moderate-risk modules scope: ci CI/CD workflows size: M 50-199 changed lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant