-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
meta: use a single .env
file for config
#3498
Conversation
Fixes: transloadit#3473 Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com> Co-authored-by: Mikael Finstad <finstaden@gmail.com>
@@ -25,7 +25,7 @@ import DropTarget from '@uppy/drop-target' | |||
import Audio from '@uppy/audio' | |||
/* eslint-enable import/no-extraneous-dependencies */ | |||
|
|||
// DEV CONFIG: create a .env.local file to customize those values. | |||
// DEV CONFIG: create a .env file in the project root directory to customize those values. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice 👍 i forgot about that
set +a | ||
nodemon --watch packages/@uppy/companion/src --exec node ./packages/@uppy/companion/src/standalone/start-server.js | ||
if [ -f .env ]; then | ||
nodemon --watch packages/@uppy/companion/src --exec node -r dotenv/config ./packages/@uppy/companion/src/standalone/start-server.js |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice trick. did you test this locally with npm run start:companion?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes looks like it's working
Co-authored-by: Mikael Finstad <finstaden@gmail.com>
| Package | Version | Package | Version | | --------------- | ------- | --------------- | ------- | | @uppy/companion | 3.3.0 | uppy | 2.6.0 | | @uppy/robodog | 2.3.2 | | | - meta: warn about not merging PR manually (Artur Paikin / #3492) - @uppy/companion: fix unpslash author meta, sanitize metadata to strings and improve companion tests (Mikael Finstad / #3478) - meta: ensure README is correctly formatted when doing releases (Antoine du Hamel / #3499) - meta: fix CDN bundle (Antoine du Hamel / #3494) - meta: fix missing EOL and end of e2e test templates (Antoine du Hamel / #3484) - meta: use a single `.env` file for config (Antoine du Hamel / #3498)
| Package | Version | Package | Version | | --------------- | ------- | --------------- | ------- | | @uppy/companion | 3.3.0 | uppy | 2.6.0 | | @uppy/robodog | 2.3.2 | | | - meta: warn about not merging PR manually (Artur Paikin / transloadit#3492) - @uppy/companion: fix unpslash author meta, sanitize metadata to strings and improve companion tests (Mikael Finstad / transloadit#3478) - meta: ensure README is correctly formatted when doing releases (Antoine du Hamel / transloadit#3499) - meta: fix CDN bundle (Antoine du Hamel / transloadit#3494) - meta: fix missing EOL and end of e2e test templates (Antoine du Hamel / transloadit#3484) - meta: use a single `.env` file for config (Antoine du Hamel / transloadit#3498)
Fixes: #3473
Refs: #3478 (comment)