-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
Update browser-ui-test version to 0.8.3 #95013
Conversation
I think your NPM package is broken. The |
Indeed, great catch! I'll update the PR once I have fixed this. |
4d680ea
to
45a3075
Compare
Again, thanks a lot @notriddle! I added a second commit which will run GUI test if browser-ui-test version is updated to ensure that this issue will never happen. |
@bors r+ rollup=always |
📌 Commit 45a3075 has been approved by |
🌲 The tree is currently closed for pull requests below priority 100. This pull request will be tested once the tree is reopened. |
…askrgr Rollup of 4 pull requests Successful merges: - rust-lang#95013 (Update browser-ui-test version to 0.8.2) - rust-lang#95039 (Make negative coherence work when there's impl negative on super predicates) - rust-lang#95047 (Refactor: remove an unnecessary pattern for ignoring all parts) - rust-lang#95048 (update Miri) Failed merges: r? `@ghost` `@rustbot` modify labels: rollup
It brings mostly debugging improvements: it doesn't stop at the first failing command but rather at the first "fatal error".
r? @notriddle