-
Notifications
You must be signed in to change notification settings - Fork 916
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
docs: update docs to reflect is-ignored changes #631
Conversation
As per #595 (comment) |
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.
Some minor nitpicking, but overall looks great!
docs/reference-configuration.md
Outdated
/* | ||
* If this is true we will not use any of the default is-ignored rules | ||
* Wether the default ignore rules should be used |
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.
I would end here with a period too since it's a sentence (and you did that for the upper one).
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.
I also checked it with Grammarly, it's stating that it's not Wether
but Whether
. But I'm Dutch and have no clue about proper spelling 😄
docs/reference-configuration.md
Outdated
(commit) => commit === '' | ||
], | ||
/* | ||
* If this is true then the default ignores like `Merge commit` are not ignored | ||
* If this is false then the default ignores like `Merge commit` are not ignored |
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.
Same here, I would personally end with a period too.
Using Grammarly is a good idea for docs texts. Rephrased the docs to be a bit easier to read. |
No description provided.