Skip to content

chore: remove leftover implied reference to create default location ff#7499

Merged
vincentsalucci merged 2 commits intomainfrom
ac/leftover-test-ff-ref
Apr 17, 2026
Merged

chore: remove leftover implied reference to create default location ff#7499
vincentsalucci merged 2 commits intomainfrom
ac/leftover-test-ff-ref

Conversation

@vincentsalucci
Copy link
Copy Markdown
Member

@vincentsalucci vincentsalucci commented Apr 17, 2026

📔 Objective

Removed indirect reference to create-default-location feature flag that doesn't exist anymore

@vincentsalucci vincentsalucci requested a review from eliykat April 17, 2026 21:16
@vincentsalucci vincentsalucci self-assigned this Apr 17, 2026
@vincentsalucci vincentsalucci enabled auto-merge (squash) April 17, 2026 21:17
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 17, 2026

Logo
Checkmarx One – Scan Summary & Detailsb021f205-f7b6-45a4-82d3-25018568c33d

Great job! No new security vulnerabilities introduced in this pull request

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 17, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.11%. Comparing base (4cc7fb4) to head (7ed5c2f).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7499   +/-   ##
=======================================
  Coverage   59.11%   59.11%           
=======================================
  Files        2078     2078           
  Lines       91535    91535           
  Branches     8142     8142           
=======================================
+ Hits        54110    54111    +1     
+ Misses      35497    35496    -1     
  Partials     1928     1928           

☔ 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.

@vincentsalucci vincentsalucci requested a review from a team as a code owner April 17, 2026 22:00
@sonarqubecloud
Copy link
Copy Markdown

@vincentsalucci vincentsalucci merged commit cd4e44c into main Apr 17, 2026
43 checks passed
@vincentsalucci vincentsalucci deleted the ac/leftover-test-ff-ref branch April 17, 2026 23:10
@djsmith85 djsmith85 added the t:tech-debt Change Type - Tech debt label May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t:tech-debt Change Type - Tech debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants