Skip to content
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: Update snapshot tests for Textual 0.77 #671

Merged

Conversation

godlygeek
Copy link
Contributor

The new version of Textual adds a key to the footer for opening the command palette. We could disable this, since it's taking up a bit of precious screen real estate, but one of the options in the command palette is a full list of keys bindings that can be opened in a side panel. That's useful enough to justify keeping this item, even if it means cutting off our own bindings.

The new version of Textual adds a key to the footer for opening the
command palette. We could disable this, since it's taking up a bit of
precious screen real estate, but one of the options in the command
palette is a full list of keys bindings that can be opened in a side
panel. That's useful enough to justify keeping this item, even if it
means cutting off our own bindings.

Signed-off-by: Matt Wozniski <[email protected]>
@godlygeek godlygeek self-assigned this Aug 26, 2024
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.01%. Comparing base (58adb86) to head (511f2bf).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #671      +/-   ##
==========================================
+ Coverage   92.98%   93.01%   +0.02%     
==========================================
  Files          94       94              
  Lines       11445    11445              
  Branches     2114     2114              
==========================================
+ Hits        10642    10645       +3     
+ Misses        803      800       -3     
Flag Coverage Δ
cpp 93.01% <ø> (+0.02%) ⬆️
python_and_cython 93.01% <ø> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@pablogsal pablogsal merged commit 8ab133b into bloomberg:main Aug 27, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants