Skip to content
This repository was archived by the owner on Oct 22, 2024. It is now read-only.

chore: split app and extension environment variables#444

Merged
matthewwalsh0 merged 10 commits intomainfrom
chore/336-env
Jan 20, 2023
Merged

chore: split app and extension environment variables#444
matthewwalsh0 merged 10 commits intomainfrom
chore/336-env

Conversation

@matthewwalsh0
Copy link
Copy Markdown
Member

@matthewwalsh0 matthewwalsh0 commented Jan 19, 2023

Overview

Split the app and extension environment variables so the app uses a standard .env file and the extension continues to use the .metamaskrc file.

Changes

Root

  • Update initialisation scripts to automatically create .env if it doesn't exist.
  • Update initialisation scripts to automatically create .metamaskrc file if it doesn't exist.
  • Simplify README following above automation.

App

  • Use .env file when building app.
  • Use .env file when building UI.
  • Use .env file in Windows packaging docker configuration.
  • List all possible app environment variables with descriptions in .env.example.
  • Copy .metamaskrc.dist to packages/app and list all possible extension environment variables with descriptions.
  • Update README to avoid duplicating environment variable descriptions.
  • Replace raw env check with cfg() usage.

App E2E

  • Remove .env.test.e2e in favour of constants in new app tests helper.
  • Replace state clearing logic in app tests with use of Yarn script.
  • Create delay utility function.
  • Add delay to unstable test.

Add .metamaskrc.dist to app workspace to list all possible environment variables.
Replace app test environment variables with constants.
Use Yarn script to clear state in app tests.
Use env file in windows docker compose.
Add packaging envs to example env file.
Update UI LavaMoat policies.
@matthewwalsh0 matthewwalsh0 marked this pull request as ready for review January 20, 2023 08:43
@matthewwalsh0 matthewwalsh0 requested a review from a team January 20, 2023 08:43
@matthewwalsh0 matthewwalsh0 merged commit a222d9b into main Jan 20, 2023
@matthewwalsh0 matthewwalsh0 deleted the chore/336-env branch January 20, 2023 17:13
@cryptotavares cryptotavares mentioned this pull request Mar 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants