Skip to content

Conversation

@AdamGS
Copy link
Contributor

@AdamGS AdamGS commented Dec 18, 2025

This PR disables deployment permissions for the polar-signals action. The current deployments create a ton of visual noise in long-running PRs (#5676) which makes it really hard to follow what's actually going on.
I've added explicit deployment permission to actions that use github deployments (docs and publishing), and the polar-signals actions should still work even if it can't deploy.

@AdamGS AdamGS requested review from 0ax1 and joseph-isaacs December 18, 2025 15:46
@AdamGS AdamGS added the changelog/chore A trivial change label Dec 18, 2025
@AdamGS AdamGS force-pushed the adamg/disable-polar-signals-deploy branch from 15392f8 to 21bc977 Compare December 18, 2025 15:47
@AdamGS AdamGS changed the title Disable polar-signals deployments ci: Disable polar-signals deployments Dec 18, 2025
@AdamGS AdamGS enabled auto-merge (squash) December 18, 2025 15:49
Signed-off-by: Adam Gutglick <adam@spiraldb.com>
@AdamGS AdamGS force-pushed the adamg/disable-polar-signals-deploy branch from 21bc977 to 53cacc2 Compare December 18, 2025 15:51
@AdamGS AdamGS merged commit 397d7f4 into develop Dec 18, 2025
47 checks passed
@AdamGS AdamGS deleted the adamg/disable-polar-signals-deploy branch December 18, 2025 16:00
@codecov
Copy link

codecov bot commented Dec 18, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.85%. Comparing base (fb976a1) to head (53cacc2).
⚠️ Report is 1 commits behind head on develop.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/chore A trivial change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants