Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Quick Regression Sanity Check #10885

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

Myoldmopar
Copy link
Member

The huge list of regressions was appearing on GHA right from the start of the window manager branch. For now I just want a sanity check that I can reproduce that. Investigation will continue based on the outcome here...

@Myoldmopar Myoldmopar added the DoNotPublish Includes changes that shouldn't be reported in the changelog label Dec 30, 2024
Copy link

⚠️ Regressions detected on macos-14 for commit 9866c49

Regression Summary
  • ESO Small Diffs: 651
  • EIO: 577
  • MTR Small Diffs: 541
  • Table Small Diffs: 462
  • Table String Diffs: 196
  • ZSZ Small Diffs: 63
  • JSON Big Diffs: 3
  • Table Big Diffs: 24
  • ERR: 18
  • EDD: 5
  • MTR Big Diffs: 2
  • ESO Big Diffs: 8
  • SSZ Small Diffs: 14
  • DELightIn: 1
  • DELightOut: 1
  • JSON Small Diffs: 1

Copy link

⚠️ Regressions detected on macos-14 for commit cb3fd7e

Regression Summary
  • ESO Small Diffs: 651
  • EIO: 577
  • MTR Small Diffs: 541
  • Table Small Diffs: 462
  • Table String Diffs: 196
  • ZSZ Small Diffs: 63
  • JSON Big Diffs: 3
  • Table Big Diffs: 24
  • ERR: 18
  • EDD: 5
  • MTR Big Diffs: 2
  • ESO Big Diffs: 8
  • SSZ Small Diffs: 14
  • DELightIn: 1
  • DELightOut: 1
  • JSON Small Diffs: 1

@Myoldmopar
Copy link
Member Author

OK, things seem about as expected, now to pull in the commit that was intended to resolve diffs.

@Myoldmopar
Copy link
Member Author

Scratch that, there was one more commit to run first.

Copy link

github-actions bot commented Jan 2, 2025

⚠️ Regressions detected on macos-14 for commit e0c8a23

Regression Summary
  • ESO Small Diffs: 651
  • EIO: 577
  • MTR Small Diffs: 541
  • Table Small Diffs: 462
  • Table String Diffs: 196
  • ZSZ Small Diffs: 63
  • JSON Big Diffs: 3
  • Table Big Diffs: 24
  • ERR: 18
  • EDD: 5
  • MTR Big Diffs: 2
  • ESO Big Diffs: 8
  • SSZ Small Diffs: 14
  • DELightIn: 1
  • DELightOut: 1
  • JSON Small Diffs: 1

@Myoldmopar
Copy link
Member Author

OK, and now, with your "This was the source of the diffs? THIS?" commit pulled in, I do indeed get no diffs locally. Let's see what CI is showing.

Copy link

github-actions bot commented Jan 2, 2025

⚠️ Regressions detected on macos-14 for commit 596fa0c

Regression Summary
  • ESO Small Diffs: 650
  • MTR Small Diffs: 539
  • EIO: 555
  • Table Small Diffs: 442
  • Table String Diffs: 212
  • ZSZ Small Diffs: 50
  • JSON Big Diffs: 3
  • Table Big Diffs: 27
  • ERR: 21
  • EDD: 4
  • MTR Big Diffs: 3
  • ESO Big Diffs: 9
  • SSZ Small Diffs: 16
  • DELightIn: 1
  • DELightOut: 1
  • JSON Small Diffs: 1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DoNotPublish Includes changes that shouldn't be reported in the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants