-
Notifications
You must be signed in to change notification settings - Fork 65
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 shortcuts help dialog #400
Conversation
syg
commented
Jan 6, 2022
@bakkot The multipage baseline is failing in a way I don't quite understand. Maybe this shortcuts |
Running |
Sorry, I meant update-baselines on multipage is failing in a way I don't understand:
|
Ah. It's complaining because it sees the Presumably we want this to be on every page of the multipage build, like the table of contents and unlike the rest of the boilerplate. So, easiest fix is to move this out of |
Perfect, thanks for the help. |
Ah jeeze, now it's failing because of a baseline-like test that isn't a baseline. I can push up a fix for that one. |
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!