bos-loader icon indicating copy to clipboard operation
bos-loader copied to clipboard

Do not throw error on encountering `REPL_ACCOUNT` in replacements file

Open mpeterdev opened this issue 2 years ago • 0 comments

Originally it was decided to error out if REPL_ACCOUNT was defined in the replacements file since the value will not be used. The account supplied to serve the components under always takes precedence and I believed not throwing an error would lead to confusion where a user doesn't realize their value in the replacements file is being ignored.

This has proven to be a headache, since replacement files may be shared between bos-loader and CI but REPL_ACCOUNT must be supplied in CI. In https://github.com/near/near-discovery-components we find ourselves needing to locally delete the REPL_ACCOUNT line from the replacements file in order to run bos-loader and then revert that change before committing.

If REPL_ACCOUNT is defined in the replacements file, we should print a warning message that the value is not being used but otherwise proceed with normal execution

mpeterdev avatar Aug 16 '23 14:08 mpeterdev