dsanders11

#47303: ci: add a problem matcher for ESLint output

Merged
Created: May 29, 2025, 3:42:21 PM
Merged: May 29, 2025, 3:58:17 PM
5 comments
Target: main

Description of Change

Will catch errors like the lint failure currently on main. This is taken verbatim from https://github.com/actions/setup-node/blob/49933ea5288caeca8642d1e84afbd3f7d6820020/.github/eslint-stylish.json, which is automatically injected for jobs using actions/setup-node, but we usually use the Node.js version on our runners, and specifically our lint job does not use actions/setup-node.

Checklist

  • PR description included and stakeholders cc'd

Release Notes

Notes: none

Backports

34-x-y
In-flight
PR Number
#47307
Waiting to be merged
35-x-y
Merged
PR Number
#47306
Merged At
May 30, 2025, 4:02:04 AM
Released In
Not yet
Release Date
Not yet
36-x-y
Merged
PR Number
#47308
Merged At
May 30, 2025, 4:03:04 AM
Released In
Not yet
Release Date
Not yet
37-x-y
Merged
PR Number
#47305
Merged At
May 30, 2025, 4:00:37 AM
Released In
Not yet
Release Date
Not yet

Semver Impact

Major
Breaking changes
Minor
New features
Patch
Bug fixes
None
Docs, tests, etc.

Semantic Versioning helps users understand the impact of updates:

  • Major (X.y.z): Breaking changes that may require code modifications
  • Minor (x.Y.z): New features that maintain backward compatibility
  • Patch (x.y.Z): Bug fixes that don't change the API
  • None: Changes that don't affect using facing parts of Electron