8000
Skip to content

BUGFIX: fix removal of str if it begins an expression when minifying.#186

Merged
BrianPugh merged 1 commit intomainfrom
fix-minify
Jun 2, 2025
Merged

BUGFIX: fix removal of str if it begins an expression when minifying.#186
BrianPugh merged 1 commit intomainfrom
fix-minify

Conversation

@BrianPugh
Copy link
Copy Markdown
Owner

Minifying would break expressions that started with a string, like "a" in {'a', 'b', 'c'}. This fixes that.

@codecov
Copy link
Copy Markdown
codecov bot commented Jun 2, 2025

Codecov Report

Attention: Patch coverage is 83.33333% with 2 lines in your changes missing coverage. Please review.

Project coverage is 60.63%. Comparing base (8e7070c) to head (5af7fff).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
belay/_minify.py 83.33% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #186      +/-   ##
==========================================
+ Coverage   60.56%   60.63%   +0.07%     
==========================================
  Files          40       40              
  Lines        2087     2096       +9     
  Branches      371      373       +2     
==========================================
+ Hits         1264     1271       +7     
- Misses        730      732       +2     
  Partials       93       93              
Flag Coverage Δ
unittests 60.59% <83.33%> (+0.07%) ⬆️

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.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@BrianPugh BrianPugh merged commit 9276d2a into main Jun 2, 2025
13 checks passed
@BrianPugh BrianPugh deleted the fix-minify branch June 2, 2025 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

0