fix(cli): clarify suppress-warnings help text - #172
Merged
Conversation
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Closes #171
改动
.option( "-s, --suppress-warnings", - "suppress all warnings, that means warnings will not block CI（抑制所有警告，这意味着警告不会阻止 CI）" + "do not let warnings affect the exit code（忽略 warning 对退出码的影响）" )理由
旧文案 "suppress all warnings" 与实现不符：warning 仍然显示，只是不参与退出码判定。README 的描述一直是准确的，本 PR 让 CLI help 与 README、实际实现对齐。
验证
--help输出新文案