Skip to content

Chore todo optimization#67

Merged
ColinMichaels merged 4 commits into
devfrom
chore-todo-optimization
Feb 28, 2026
Merged

Chore todo optimization#67
ColinMichaels merged 4 commits into
devfrom
chore-todo-optimization

Conversation

@ColinMichaels
Copy link
Copy Markdown
Owner

No description provided.

…t to deployment workflows**

- Added `production` environment context to deployment jobs in `firebase_deployment_workflow.yml`, `firebase-hosting-merge.yml`, and `firebase-hosting-pull-request.yml` to ensure proper deployment environment handling.
- Implemented step to validate required environment variables in all deployment workflows:
  - Checks presence of critical environment variables such as `APP_TITLE`, `APP_API_URL`, multiple Firebase credentials, `OPENAI_API_KEY`, and `OPEN_WEATHER_MAP_API_KEY`.
  - Outputs missing variables and provides guidance for misconfiguration resolution.
  - Exits workflow for incomplete configurations, improving deployment reliability.
- Enhanced workflow consistency by introducing environment validation across all Firebase workflows.

This improves deployment robustness, prevents misconfigured environments in production, and enhances debugging clarity.

Signed-off-by: Colin Michaels <colin@colinmichaels.com>
@ColinMichaels ColinMichaels self-assigned this Feb 28, 2026
@ColinMichaels ColinMichaels merged commit 87befc7 into dev Feb 28, 2026
2 checks passed
@sonarqubecloud
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown

Visit the preview URL for this PR (updated for commit e70bd7f):

https://colinmichaels--pr67-chore-todo-optimizat-czjwfe6x.web.app

(expires Sat, 07 Mar 2026 20:25:57 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 825a61d1eb4d817c6dec41f3afc8f4be6955b0e3

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.

1 participant