Skip to content

fix: csp dev warning#15459

Merged
florian-lefebvre merged 2 commits intomainfrom
fix/csp-dev-warning
Feb 16, 2026
Merged

fix: csp dev warning#15459
florian-lefebvre merged 2 commits intomainfrom
fix/csp-dev-warning

Conversation

@florian-lefebvre
Copy link
Member

Changes

  • I recently updated context.csp to be undefined and warn when unavailable, to match sessions
  • However, that caused it to always warn in dev since we don't support CSP in dev at the moment
  • This PR skips the warning in dev

Testing

N/A

Docs

Changeset

@florian-lefebvre florian-lefebvre self-assigned this Feb 10, 2026
@changeset-bot
Copy link

changeset-bot bot commented Feb 10, 2026

🦋 Changeset detected

Latest commit: cfe875b

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added the pkg: astro Related to the core `astro` package (scope) label Feb 10, 2026
@codspeed-hq
Copy link

codspeed-hq bot commented Feb 10, 2026

Merging this PR will not alter performance

✅ 9 untouched benchmarks


Comparing fix/csp-dev-warning (cfe875b) with main (7038f07)1

Open in CodSpeed

Footnotes

  1. No successful run was found on main (da5f464) during the generation of this report, so 7038f07 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

Copy link
Member

@sarah11918 sarah11918 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a thought from me, Florian!

@florian-lefebvre florian-lefebvre merged commit a4406b4 into main Feb 16, 2026
25 checks passed
@florian-lefebvre florian-lefebvre deleted the fix/csp-dev-warning branch February 16, 2026 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pkg: astro Related to the core `astro` package (scope)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments