Skip to content

bugfix: sandbox_paths is nullable, check for it in AWS#1632

Merged
veluca93 merged 1 commit intocms-dev:mainfrom
prandla:bugfix-aws-sandbox-paths
Feb 11, 2026
Merged

bugfix: sandbox_paths is nullable, check for it in AWS#1632
veluca93 merged 1 commit intocms-dev:mainfrom
prandla:bugfix-aws-sandbox-paths

Conversation

@prandla
Copy link
Member

@prandla prandla commented Feb 11, 2026

jinja being too dynamic strikes again....

mostly this is a problem when the submission is still compiling, then both sandbox_paths and sandbox_digests are None.

@codecov
Copy link

codecov bot commented Feb 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 54.67%. Comparing base (4cdaa00) to head (d4c62cf).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1632      +/-   ##
==========================================
- Coverage   54.67%   54.67%   -0.01%     
==========================================
  Files         335      335              
  Lines       27361    27361              
==========================================
- Hits        14960    14959       -1     
- Misses      12401    12402       +1     
Flag Coverage Δ
functionaltests 0.00% <ø> (ø)
unittests 54.67% <ø> (-0.01%) ⬇️

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

@veluca93 veluca93 merged commit f2af1fb into cms-dev:main Feb 11, 2026
4 checks passed
@prandla prandla deleted the bugfix-aws-sandbox-paths branch February 14, 2026 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants