docs:fix typo #1537
Merged
docs:fix typo #1537
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
Codecov Report
@@ Coverage Diff @@
## master #1537 +/- ##
=======================================
Coverage 99.71% 99.71%
=======================================
Files 29 29
Lines 714 714
=======================================
Hits 712 712
Misses 2 2Continue to review full report at Codecov.
|
solarhell
reviewed
Oct 17, 2017
| @@ -144,7 +144,7 @@ egg 基于 [semver] 语义化版本号进行发布。 | |||
|
|
|||
| ### 分支策略 | |||
|
|
|||
| `master` 分支为当前稳定发布的版本，`next` 分支为下一个开发中的大版本。 | |||
| `master` 分支为当前稳定发布的版本，`next` 分支为下一个开发中的大版本。 | |||
|
|
|||
| - 只维护两个版本，除非有安全问题，否则修复只会 patch 到 `master` 和 `next` 分支，其他更新推动上层框架升级到稳定大版本的最新版本。 | |||
| - 所有 API 的废弃都需要在当前的稳定版本上 `deprecate` 提示，并保证在当前的稳定版本上一直兼容到新版本的发布。 | |||
There was a problem hiding this comment.
thank you for your suggestion, solarhell.
i think using deprecate or deprecated is only a personal habit. so i won't to modify it.
There was a problem hiding this comment.
deprecate is better, since egg provide app.deprecate()
|
You'd better modify english version at the same time. |
popomore
approved these changes
Oct 19, 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.
Checklist
npm testpassesAffected core subsystem(s)
Description of change