-
-
Notifications
You must be signed in to change notification settings - Fork 726
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
Add full dashboard link in scheduler logs #7631
Add full dashboard link in scheduler logs #7631
Conversation
Unit Test ResultsSee test report for an extended history of previous test failures. This is useful for diagnosing flaky tests. 26 files ± 0 26 suites ±0 12h 14m 39s ⏱️ + 1h 4m 27s For more details on these failures and errors, see this check. Results for commit 1a49a3b. ± Comparison against base commit 689b603. This pull request removes 1 and adds 8 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
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.
This is great, being able to click a link in the terminal is so helpful. Especially given that VSCode auto port forwards over SSH when you do.
Co-authored-by: Jacob Tomlinson <[email protected]>
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.
Nice -- thanks @milesgranger
Thanks for addressing the review feedback here @milesgranger. CI failures look unrelated. Merging. |
Closes #7543
pre-commit run --all-files