Skip to content

fix: prefix quick open should trim input#2172

Merged
erha19 merged 1 commit intomainfrom
fix/no-trim
Jan 10, 2023
Merged

fix: prefix quick open should trim input#2172
erha19 merged 1 commit intomainfrom
fix/no-trim

Conversation

@bytemain
Copy link
Copy Markdown
Member

@bytemain bytemain commented Jan 6, 2023

Types

  • 🐛 Bug Fixes

Background or solution

close: #1958

After:
CleanShot 2023-01-06 at 13 20 35@2x

Changelog

trim user input in prefix quickopen

@github-actions github-actions Bot added the 🐞 bug Something isn't working label Jan 6, 2023
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 6, 2023

Codecov Report

Base: 57.79% // Head: 57.79% // Decreases project coverage by -0.00% ⚠️

Coverage data is based on head (ee08b86) compared to base (7593498).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2172      +/-   ##
==========================================
- Coverage   57.79%   57.79%   -0.01%     
==========================================
  Files        1313     1313              
  Lines       82995    82995              
  Branches    17254    17254              
==========================================
- Hits        47970    47966       -4     
- Misses      31835    31838       +3     
- Partials     3190     3191       +1     
Flag Coverage Δ
jsdom 52.73% <ø> (-0.01%) ⬇️
node 16.52% <ø> (ø)

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

Impacted Files Coverage Δ
packages/monaco/src/browser/schema-registry.ts 68.88% <0.00%> (-2.23%) ⬇️
...kages/theme/src/browser/workbench.theme.service.ts 71.38% <0.00%> (-0.58%) ⬇️
...erences/src/browser/preference-settings.service.ts 76.41% <0.00%> (-0.44%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@erha19 erha19 merged commit b4dec99 into main Jan 10, 2023
@erha19 erha19 deleted the fix/no-trim branch January 10, 2023 02:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] 命令匹配没有 trim 前后多余的空格

2 participants