Skip to content

CATALYST-1744 - Remove default loading of environment variable files#2877

Merged
jamesqquick merged 2 commits intoalphafrom
CATALYST-1744-remove-default-env-file
Feb 16, 2026
Merged

CATALYST-1744 - Remove default loading of environment variable files#2877
jamesqquick merged 2 commits intoalphafrom
CATALYST-1744-remove-default-env-file

Conversation

@jamesqquick
Copy link
Copy Markdown
Contributor

What/Why?

Remove the reading of default environment variable files.

Testing

pnpm build --filter @bigcommerce/catalyst compiles without errors
pnpm --filter @bigcommerce/catalyst test — all tests pass

TO manually test. First, delete an existing .bigcommerce/project.json file if it exists. Then, show that running catalyst project create will not work if you don't pass variables in explicit flags.

Migration

Pass environment variable files explicitly using flags in the command or include them in your project.json

@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 11, 2026

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

Project Deployment Actions Updated (UTC)
catalyst Ready Ready Preview, Comment Feb 16, 2026 3:29pm

Request Review

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Feb 11, 2026

🦋 Changeset detected

Latest commit: 63b578f

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

This PR includes changesets to release 1 package
Name Type
@bigcommerce/catalyst Patch

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

@jamesqquick jamesqquick force-pushed the CATALYST-1744-write-variables-to-project-config branch 2 times, most recently from 0ff8bd7 to fd93337 Compare February 11, 2026 20:29
@jamesqquick jamesqquick force-pushed the CATALYST-1744-remove-default-env-file branch from 44a100f to 6691755 Compare February 11, 2026 21:30
@jamesqquick jamesqquick changed the base branch from CATALYST-1744-write-variables-to-project-config to alpha February 11, 2026 21:30
@jamesqquick jamesqquick changed the base branch from alpha to CATALYST-1744-write-variables-to-project-config February 11, 2026 21:30
@jamesqquick jamesqquick force-pushed the CATALYST-1744-remove-default-env-file branch from 6691755 to 7b16ea2 Compare February 11, 2026 21:31
@jamesqquick jamesqquick force-pushed the CATALYST-1744-remove-default-env-file branch from 7b16ea2 to b9bb8b0 Compare February 11, 2026 21:34
@jamesqquick jamesqquick changed the base branch from CATALYST-1744-write-variables-to-project-config to alpha February 11, 2026 21:34
@jamesqquick jamesqquick marked this pull request as ready for review February 11, 2026 21:37
@jamesqquick jamesqquick requested a review from a team February 11, 2026 21:37
Copy link
Copy Markdown
Contributor

@matthewvolk matthewvolk left a comment

Choose a reason for hiding this comment

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

Just one comment

Comment thread packages/catalyst/src/cli/program.ts
@jamesqquick jamesqquick merged commit da9e7de into alpha Feb 16, 2026
8 checks passed
@jamesqquick jamesqquick deleted the CATALYST-1744-remove-default-env-file branch February 16, 2026 16:06
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.

3 participants