Thanks to visit codestin.com
Credit goes to github.com

Skip to content

ci: run ecosystem tests on main branch#20891

Open
sethamus wants to merge 1 commit into
eslint:mainfrom
sethamus:ecosystem-tests-on-main
Open

ci: run ecosystem tests on main branch#20891
sethamus wants to merge 1 commit into
eslint:mainfrom
sethamus:ecosystem-tests-on-main

Conversation

@sethamus
Copy link
Copy Markdown
Contributor

Prerequisites checklist

AI acknowledgment

  • I did not use AI to generate this PR.
  • (If the above is not checked) I have reviewed the AI-generated content before submitting.

What is the purpose of this pull request? (put an "X" next to an item)

[ ] Documentation update
[ ] Bug fix (template)
[ ] New rule (template)
[ ] Changes an existing rule (template)
[ ] Add autofix to a rule
[ ] Add a CLI option
[ ] Add something to the core
[x] Other, please explain:

This PR progresses to step 2 of the ecosystem tests rollout plan.

What changes did you make? (Give an overview)

  • Added main to push.branches so the workflow runs on every push to main
  • Removed continue-on-error: true so test failures are reported as real build failures
  • Updated the Discord notification condition to use failure() instead of steps.tester.outcome

Refs #20819

Is there anything you'd like reviewers to focus on?

@sethamus sethamus requested a review from a team as a code owner May 19, 2026 13:50
@github-project-automation github-project-automation Bot moved this to Needs Triage in Triage May 19, 2026
@eslint-github-bot eslint-github-bot Bot added the build This change relates to ESLint's build process label May 19, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented May 19, 2026

Deploy Preview for docs-eslint canceled.

Name Link
🔨 Latest commit 8a00f60
🔍 Latest deploy log https://app.netlify.com/projects/docs-eslint/deploys/6a0c6ac1bd824e000733779a

@lumirlumir lumirlumir added the accepted There is consensus among the team that this change meets the criteria for inclusion label May 20, 2026
@lumirlumir lumirlumir moved this from Needs Triage to Second Review Needed in Triage May 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

accepted There is consensus among the team that this change meets the criteria for inclusion build This change relates to ESLint's build process github actions

Projects

Status: Second Review Needed

Development

Successfully merging this pull request may close these issues.

3 participants