MetaMask / MetaMask/metamask-extension

`process.env.*` vars should be read once at the top of the file

Open
#22,831 0 comments 0 reactions 0 assignees View on GitHub
INVALID-ISSUE-TEMPLATE
Dominant language
TypeScript
Stars
13.2k
Forks
5.6k
Avg merge
2d 5h
Merged PRs (30d)
451

Description

### What is this about?

> we may be doing it elsewhere but as a point of style I like to read the ENV at the beginning of the module, and use that state elsewhere

_Originally posted by @kumavis in https://github.com/MetaMask/metamask-extension/pull/22524#discussion_r1480352595_

Contributor guide

Open the contributing guide

Research direction

No files or tests are named. Search the TypeScript sources for process.env.* references and review how each module currently reads them. Done means the relevant environment variables are read once at the top of each affected module and reused, with the existing test suite still passing.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
tooling
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.