dsanders11

#49701: ci: fix patches changes detected in apply patches workflow

Merged
Created: Feb 6, 2026, 1:21:33 AM
Merged: Feb 6, 2026, 3:48:31 PM
4 comments
Target: main

Description of Change

Fast follow-up to #49667, because "It leaves the changes staged rather than committing them since it's just for checking purposes." turned out to be famous last words.

I didn't account for the part of our CI workflow which tries to detect patch changes by re-exporting them after sync, which was flagging the staged changes as a change to a patch and failing out. This was not caught originally when testing #49667 as there was a cache match (expected) so the sync never took place. This PR has been tested with the cache disabled to ensure a more complete E2E test.

Checklist

  • PR description included

Release Notes

Notes: none

Backports

39-x-y
Merged
PR Number
#49709
Merged At
Feb 6, 2026, 11:29:56 PM
Released In
Not yet
Release Date
Not yet
40-x-y
Merged
PR Number
#49710
Merged At
Feb 6, 2026, 11:29:53 PM
Released In
Not yet
Release Date
Not yet
41-x-y
Merged
PR Number
#49708
Merged At
Feb 6, 2026, 11:30:15 PM
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