Skip to content
This repository has been archived by the owner on Aug 7, 2023. It is now read-only.

Set maxComplexity default to -1 with a description #129

Merged
merged 1 commit into from
Dec 16, 2015
Merged

Set maxComplexity default to -1 with a description #129

merged 1 commit into from
Dec 16, 2015

Conversation

toxinu
Copy link

@toxinu toxinu commented Dec 16, 2015

Pull request for issue #121.
I just add a simple description for this settings with a new default value to disable it.

😎

@@ -120,8 +120,9 @@ module.exports =
items:
type: 'string'
maxComplexity:
description: 'McCabe complexity threshold (`-1` to disable it)'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd just remove the "it" unless you want to expand on what "it" is.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just rebase my branch with fixed commit.

Arcanemagus added a commit that referenced this pull request Dec 16, 2015
Set maxComplexity default to -1 with a description
@Arcanemagus Arcanemagus merged commit 2995d66 into AtomLinter:master Dec 16, 2015
@toxinu toxinu deleted the 121-max-complexity branch December 16, 2015 20:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants