-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Tests fail on all recent PRs #10986
Comments
I'll take a look and see if I can figure it out. |
It appears to me that some tests still fail consistently, i.e. the Now I wonder if these PR's are still candidates/evaluated for merging (???). I have created #13927, and I see no reason why it should have broken anything. |
@dalanicolai I don't think that your PR is causing these problems in CI and we are seeing the two mentioned PRs as candidates for merging. However #13907 is a draft release, therefore I have not checked it. I have read through #13927 already though I am not yet totally clear about what behaviour you have changed. I don't know why the mentioned test fails, but I think @JAremko knows about these already. |
Yeah we added more tests and found more bugs... I'm thinking about disabling some tests in PR testing and such, for now. But it kinda stinks. 🤔 |
Thanks for the replies! @smile13241324 I will elaborate on the (small but very useful) changes in #13927 in the conversation of the PR itself. |
Description
I have changed some details on the "org" mode documentation and noticed that some tests
do fail consistently for all recent PRs.
Namely these are:
FAILED test-configure-packages-2--excluded-package-is-not-configured
FAILED test-configure-packages-2--lazy-install-package-is-not-configured
FAILED test-configure-packages-2--local-package-w/-bad-string-location-gives-warning
FAILED test-configure-packages-2--local-package-w/-dotfile-owner-update-load-path
FAILED test-configure-packages-2--local-package-w/-layer-owner-update-load-path
FAILED test-configure-packages-2--local-package-w/-string-location-update-load-path
FAILED test-configure-packages-2--local-package-w/o-owner-doesnt-update-load-path
FAILED test-configure-packages-2--package-owned-by-dotfile-is-not-configured
FAILED test-configure-packages-2--package-w/-layer-owner-is-configured
FAILED test-configure-packages-2--package-w/o-owner-is-not-configured
FAILED test-configure-packages-2--protected-excluded-package-is-configured
FAILED test-configure-packages-2--protected-package-is-configured
FAILED test-configure-packages-2--site-package-is-configured
FAILED test-configure-packages-2--toggle-nil-is-not-configured
FAILED test-configure-packages-2--toggle-t-is-configured
Reproduction guide 🪲
Observed behaviour: 👀 💔
Tests fail on all recent branches
Expected behaviour: ❤️ 😄
Tests only fail if there is an issue with the contribution
System Info 💻
The text was updated successfully, but these errors were encountered: