docs: translate basic/config.md #875
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@Brian175 where are you proofreading the translation? |
Codecov Report
@@ Coverage Diff @@
## master #875 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 28 28
Lines 667 667
=====================================
Hits 667 667Continue to review full report at Codecov.
|
|
@popomore hello, this translation is from swiftgg and we have proofreaded and finalized to some degree to gurantee the quality |
dead-horse
reviewed
May 9, 2017
docs/source/en/basics/config.md
Outdated
| }; | ||
| ``` | ||
| The build-in appInfo contains: | ||
| <table> |
popomore
reviewed
May 10, 2017
docs/source/en/basics/config.md
Outdated
| |- config.unittest.js | ||
| |- config.local.js | ||
| ``` | ||
| ```config.default.js``` is the default file for configuration, and all environments will load this file. Besides, this is usually used as default configuration file for development environment. |
There was a problem hiding this comment.
inline code can change to single `
atian25
requested changes
May 10, 2017
docs/source/en/basics/config.md
Outdated
| @@ -0,0 +1,188 @@ | |||
| title: Configuration | |||
| --- | |||
| This framework provides powerful and extensible configuration function, including automatically merging applications, plug-ins, and framework's configuration. In addition, it allows users to overwrite configuration in sequence and maintain different configs depending on different environments. The result (i.e. merged config) can be accessed from ```app.config ```. | |||
docs/source/en/basics/config.md
Outdated
| 2. Using platform to manage configurations: you can pass the current environment configuration via environment variables while starting. This is a relatively elegant approach with higher requirement on operation and support from configuration platform. Moreover, The configuration environment has same flaws as first method. | ||
| 3. Using code to manage configurations: you can add some environment configurations in codes and pass them to current environment arguments while starting. However, it doesn't allow you to configurate globally and you need to alter your code whenever you want to change the configuration. | ||
|
|
||
| we choose the last method, namely **configurate with code**, The change of configuration should be also released after review. The application package itself is capable to be deployed in several environments, only by specifing the running environment. |
There was a problem hiding this comment.
method 换为 way 或 strategy 会不会好一点?
docs/source/en/basics/config.md
Outdated
| ``` | ||
| As demonstrated above, the framework overwirte arrays instead of merging them. | ||
|
|
||
| ## plug-in Configuration |
docs/source/en/basics/config.md
Outdated
| }; | ||
| ``` | ||
|
|
||
| ### Importing plugins |
There was a problem hiding this comment.
how about 'load' or 'use'
ps: Is there any suggestions for the case sensitivity of subTitles?
popomore
approved these changes
May 11, 2017
|
cc @atian25 |
atian25
approved these changes
May 16, 2017
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
'config.md' is translated by @jwssdwed and proofread by @Brian175 and finalized by @mmoaay