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
refactor: Remove grunt #2354
refactor: Remove grunt #2354
Conversation
9a1d829
to
d2ab630
Compare
6628895
to
472916c
Compare
"environment": "internal", | ||
"legalUrl": "https://wire.com/legal/", | ||
"licensesUrl": "https://wire.com/legal/licenses/", | ||
"maximumAccounts": "3", |
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.
Let's put this as a number instead of a string
"maximumAccounts": "3", | |
"maximumAccounts": 3, |
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.
Since it will be replaced with a string when loading a configuration via process.env
, I figured we should save it as a string here, too.
Breaking changes:
wrap/dist
(instead ofwrap/prod
,wrap/internal
, ...)info.json
is nowwire.json
BUILD_ENV
is nowAPP_ENV