IBM icon indicating copy to clipboard operation
IBM copied to clipboard

Add second person rule

Open ChrisChinchilla opened this issue 4 years ago • 7 comments

ChrisChinchilla avatar Apr 10 '20 12:04 ChrisChinchilla

I agree, we should use existence to check the usage of "I" and "we".

gaurav-nelson avatar Apr 16 '20 00:04 gaurav-nelson

Hmm, my reasoning was for more of a check on consistency, so the predominance of using "I" in a doc, instead of "we". If we just check for existence, then it's not so useful without context, which is not something Vale can do.

Unless I am misunderstanding what you both mean, I kind of see how both could do the same, but the consistency part was my reasoning.

ChrisChinchilla avatar Apr 22 '20 15:04 ChrisChinchilla

I think I understand your intent better now. The part of the description that says

"I" in single-author articles or "we" in multiple-author articles

could indeed be a candidate for consistency since an article can't be both single- and multi-author. (As written, though, the rule involves "we" and "you"; not "we" and "I" ...?)

If we just check for existence, then it's not so useful without context, which is not something Vale can do.

IMO, this is an ideal case for a sub-error level rule (which can be selectively disabled): It alerts writers to the fact that, according to the style guide, that they're only supposed to use "I" and "we" in certain contexts without explicitly telling them that they're doing something "wrong" (which they might not be). The alternative (limiting ourselves to precise, error-candidate rules), means we'll end up covering much less of the style guide.


Basically, from my understanding of the discussion so far, we're looking at the two following rules:

extends: consistency
message: 'Use "I" in single-author articles or "we" in multiple-author articles.'
level: error
link: https://www.ibm.com/developerworks/library/styleguidelines/#N10050
either:
  I: we 

and

extends: existence
message: 'Only use "%s" when referring to yourself, the author(s).'
level: warning
link: https://www.ibm.com/developerworks/library/styleguidelines/#N10050
tokens:
    - I
    - we

Does this seem right?

jdkato avatar Apr 22 '20 18:04 jdkato

Huh, interesting, I think I need to try implementing to see if it matches, but makes sense.

Is "sub-error level rule" an actual thing @jdkato or did you just use that as a description?

ChrisChinchilla avatar May 01 '20 12:05 ChrisChinchilla

By "sub-error" I meant warning or suggestion, which don't result in non-zero return codes.

jdkato avatar May 01 '20 17:05 jdkato

By "sub-error" I meant warning or suggestion, which don't result in non-zero return codes.

Ah ha, OK, yes, I tend to err on that side of caution for "warnings" anyway.

ChrisChinchilla avatar May 11 '20 13:05 ChrisChinchilla

OK, reminded to get back to this PR

ChrisChinchilla avatar Jul 30 '20 16:07 ChrisChinchilla