-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
test(versioning/ubuntu): Fix test failing due to new release rolling out #15210
test(versioning/ubuntu): Fix test failing due to new release rolling out #15210
Conversation
96b3713
to
0bdd907
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mock time in tests, we already do it on node versioning for that same reason
|
1096766
to
e595f68
Compare
I think this might be cleaner, as its using a build in functionality and is used as a test wrapper in |
added // istanbul ignore next: unreachable as we dont have unreleased version in out distro data file aaaand fixed the typo ^^ 😂 |
506505b
to
4a51e35
Compare
Head branch was pushed to by a user without write access
🎉 This PR is included in version 32.27.1 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Changes
Freeze isStable tests.
Context
Closes #15216
Documentation (please check one with an [x])
How I've tested my work (please tick one)
I have verified these changes via: