Skip to content

feat: support .env file#2

Open
fawni wants to merge 4 commits into
rushiiMachine:masterfrom
fawni:support_dotenv
Open

feat: support .env file#2
fawni wants to merge 4 commits into
rushiiMachine:masterfrom
fawni:support_dotenv

Conversation

@fawni

@fawni fawni commented Sep 22, 2022

Copy link
Copy Markdown

used kankyo because it's waaay lighter than dotenv or dotenvy
potentially could remove instructions on setting up environment variables manually here, but that's up to you

@rushiiMachine

rushiiMachine commented Sep 22, 2022

Copy link
Copy Markdown
Owner

Making it load .env directly will interfere with git so make it refer to .env.local instead

obscurity added 2 commits September 22, 2022 22:55
workflow should be to edit .env.local and let docker deal with the renaming
Comment thread .gitignore
/.pnp
.pnp.js
pnpm-debug.log*
.env

@rushiiMachine rushiiMachine Sep 30, 2022

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Husk I meant:

  1. keep .env as a sample file
  2. document the user copying .env to .env.local
  3. Load from .env.local

This makes .env is reserved in git as an example file

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

wrong

@rushiiMachine

Copy link
Copy Markdown
Owner

nop explode i will fix myself

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.

2 participants