Skip to content

Environment variable audit#3163

Open
laurakwhit wants to merge 2 commits intomainfrom
audit-env-vars
Open

Environment variable audit#3163
laurakwhit wants to merge 2 commits intomainfrom
audit-env-vars

Conversation

@laurakwhit
Copy link
Collaborator

Description & motivation 💭

Removes unused TEMPORAL_BANNER_TEXT and TEMPORAL_NOTIFY_ON_NEW_VERSION environment variables.

Screenshots (if applicable) 📸

Design Considerations 🎨

Testing 🧪

How was this tested 👻

  • Manual testing
  • E2E tests added
  • Unit tests added

Steps for others to test: 🚶🏽‍♂️🚶🏽‍♀️

Checklists

Draft Checklist

Merge Checklist

Issue(s) closed

DT-3639

Docs

Any docs updates needed?

@laurakwhit laurakwhit requested review from a team and Alex-Tideman as code owners February 17, 2026 21:55
@vercel
Copy link

vercel bot commented Feb 17, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
holocene Ready Ready Preview, Comment Feb 17, 2026 9:55pm

Request Review

@temporal-cicd
Copy link
Contributor

temporal-cicd bot commented Feb 17, 2026

Warnings
⚠️

📊 Strict Mode: 2 errors in 2 files (0.2% of 1122 total)

src/lib/services/settings-service.ts (1)
  • L73:2: Type '{ auth: { enabled: boolean; options: string[]; }; baseUrl: string | null; codec: { endpoint: string; passAccessToken: boolean | undefined; includeCredentials: boolean | undefined; customErrorMessage: { ...; }; }; ... 16 more ...; version: string; }' is not assignable to type 'Settings'.
src/lib/svelte-mocks/app/stores.ts (1)
  • L16:4: Type 'null' is not assignable to type 'string[]'.

Generated by 🚫 dangerJS against d629f47

@laurakwhit laurakwhit linked an issue Feb 17, 2026 that may be closed by this pull request
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.

Banner doesn't exist

2 participants

Comments