Skip to content

908: Made .env variables higher priority than .env.shared#921

Merged
Arshadul-Monir merged 1 commit into
mainfrom
908-3
Apr 27, 2026
Merged

908: Made .env variables higher priority than .env.shared#921
Arshadul-Monir merged 1 commit into
mainfrom
908-3

Conversation

@Arshadul-Monir

@Arshadul-Monir Arshadul-Monir commented Apr 26, 2026

Copy link
Copy Markdown
Contributor

908

Description of changes

Made .env the default env variables rather than .env.shared when running just backend-dev.

Checklist before review

  • I have done a thorough self-review of the PR
  • Copilot has reviewed my latest changes, and all comments have been fixed and/or closed.
  • If I have made database changes, I have made sure I followed all the db repo rules listed in the wiki here. (check if no db changes)
  • All tests have passed
  • I have successfully deployed this PR to staging
  • I have done manual QA in both dev (and staging if possible) and attached screenshots below.

Screenshots

Dev

Staging

@Arshadul-Monir

Copy link
Copy Markdown
Contributor Author

/deploy

@github-actions

Copy link
Copy Markdown
Contributor

Available PR Commands

  • /ai - Triggers all AI review commands at once
  • /review - AI review of the PR changes
  • /describe - AI-powered description of the PR
  • /improve - AI-powered suggestions
  • /deploy - Deploy to staging

See: https://github.com/tahminator/codebloom/wiki/CI-Commands

github-actions[bot]
github-actions Bot previously approved these changes Apr 26, 2026
@github-actions

Copy link
Copy Markdown
Contributor

PR Code Suggestions ✨

No code suggestions found for the PR.

@github-actions

Copy link
Copy Markdown
Contributor

Failed to generate code suggestions for PR

@Arshadul-Monir

Copy link
Copy Markdown
Contributor Author

/deploy

@Arshadul-Monir

Copy link
Copy Markdown
Contributor Author

/deploy

@github-actions

Copy link
Copy Markdown
Contributor

Failed to generate code suggestions for PR

@Arshadul-Monir Arshadul-Monir enabled auto-merge (rebase) April 26, 2026 21:48
@Arshadul-Monir Arshadul-Monir changed the title 908: Commented out DATABSE_USER and DATABASE_PASSWORD 908: Made .env variables higher priority than .env.shared Apr 27, 2026
@Arshadul-Monir Arshadul-Monir merged commit a13d9c4 into main Apr 27, 2026
64 of 74 checks passed
@Arshadul-Monir Arshadul-Monir deleted the 908-3 branch April 27, 2026 01:58
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