Skip to content

Conversation

ryan-pratt
Copy link
Contributor

These were getting skipped because appliedSettings is calculated in Widget.js' created(), so SpacerWidget always just got the default flex styling since v6.0.0

@ryan-pratt ryan-pratt requested a review from jmthomas June 9, 2025 17:56
Copy link

sonarqubecloud bot commented Jun 9, 2025

Copy link

codecov bot commented Jun 9, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.82%. Comparing base (569ef64) to head (40545de).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2122      +/-   ##
==========================================
+ Coverage   79.76%   79.82%   +0.05%     
==========================================
  Files         633      633              
  Lines       48021    47968      -53     
  Branches      754      754              
==========================================
- Hits        38304    38289      -15     
+ Misses       9634     9596      -38     
  Partials       83       83              
Flag Coverage Δ
python 83.84% <ø> (+0.04%) ⬆️
ruby-api 78.69% <ø> (ø)
ruby-backend 83.08% <ø> (+0.09%) ⬆️

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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ryan-pratt ryan-pratt merged commit 15bc996 into main Jun 9, 2025
29 checks passed
@ryan-pratt ryan-pratt deleted the bug/spacer-widget branch June 9, 2025 19:12
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.

2 participants