-
-
Notifications
You must be signed in to change notification settings - Fork 10.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
add APP_ID env. #2067
add APP_ID env. #2067
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2067 +/- ##
============================================
+ Coverage 49.96% 49.97% +<.01%
- Complexity 1978 1979 +1
============================================
Files 401 401
Lines 12406 12411 +5
Branches 1278 1279 +1
============================================
+ Hits 6199 6202 +3
- Misses 5748 5750 +2
Partials 459 459
Continue to review full report at Codecov.
|
.../main/java/com/ctrip/framework/foundation/internals/provider/DefaultApplicationProvider.java
Outdated
Show resolved
Hide resolved
.../main/java/com/ctrip/framework/foundation/internals/provider/DefaultApplicationProvider.java
Outdated
Show resolved
Hide resolved
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.
LGTM
support setting app id from OS environment variable APP_ID
No description provided.