m/metamask-extension
1
0
Fork 0
mirror of https://github.com/kremalicious/metamask-extension.git synced 2024-12-23 09:52:26 +01:00
Code

Made builds.yml variable errors more helpful (#19066)

Browse Source
This commit is contained in:
Olaf Tomalka 2023-05-09 21:47:42 +02:00 committed by GitHub
parent 6e180af313
commit a1ff530430
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
development/lib/variables.js
View File

@ -29,7 +29,11 @@ class Variables {
assert( assert(
value !== DeclaredOnly, value !== DeclaredOnly,
new TypeError( new TypeError(
`Tried to access a declared, but not defined environmental variable "${key}"`, `Tried to access a declared, but not defined environmental variable "${key}"
\tWhy am I seeing this: "${key}" is declared in builds.yml, but had no actual value when we tried loading it.
\tHow do I fix this: You could provide a default value for the variable in builds.yml under "env" property and commit to git. For example:
\t\tenv:
\t\t - ${key}: ''`,
), ),
); );
return value; return value;
@ -46,7 +50,9 @@ class Variables {
assert( assert(
this.isDeclared(key), this.isDeclared(key),
new TypeError( new TypeError(
`Tried to access an environmental variable "${key}" that wasn't declared in builds.yml`, `Tried to access an environmental variable "${key}" that wasn't declared in builds.yml
\tWhy am I seeing this: We've made use of new variables be explicit to keep track of all of them in one place
\tHow do I fix this: Adding your variable in builds.yml under "env" property and committing to git will fix this`,
), ),
); );
return this.#definitions.get(key); return this.#definitions.get(key);