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

dotenv dependency missing + using deprecated alias load #56

Open
doramatadora opened this issue Jan 23, 2020 · 0 comments
Open

dotenv dependency missing + using deprecated alias load #56

doramatadora opened this issue Jan 23, 2020 · 0 comments

Comments

@doramatadora
Copy link
Contributor

This package requires dotenv. Some teams (Internal Products, Reliability Engineering) do not have dotenv in their shared tooling. Example of failing CI build / error.
image

The current implementation also requires that dotenv be <7, because it uses the deprecated alias 'load': https://github.com/Financial-Times/fastly-tools/blob/master/tasks/deploy.js#L23 (instead of config).

See error for ^7 & the dotenv changelog: https://github.com/motdotla/dotenv/blob/master/CHANGELOG.md#700---2019-03-12

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant