-
Notifications
You must be signed in to change notification settings - Fork 179
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
.proselintrc usage #491
Comments
Adding e.g.
to |
Not sure what's going on for me. Doing:
In |
You're right. Hmm. This works:
But this doesn't:
No idea what's up. Bug? |
Yeah, I think it may be a bug. Either the IDs of these checks aren't accurate, or the parsing of this one isn't being done or applied properly. |
@cpitclaudel @lucascosti is this still a case ? wanna to confirm before trying to fix it :-) |
Sorry, I don't use proselint anymore. |
@netflash It's been a few months since I used proselint, but it was still happening back then. |
Looks like the filter in the |
I just installed proselint HEAD to try it out with my Markdown files (Python 2.7.12, Ubuntu 16.04.2), for which a lot of the default suggestions are inappropriate (eg all the Unicode stuff which Pandoc does already), and was puzzled why the |
One workaround I am trying is simply redirecting the default config file to a customized version of it:
If it can only read one config file, it'll read mine (which will work as long as I never reinstalled or upgrade, which would overwrite or ignore the symlink). This appears to work, but I've run into another logical issue: it appears that config keys go from specific to general and clash, so you cannot enable/disable members of a general category. For example, the default
Change to:
What if you want to enable specific members of the disabled category? Uh... Bad stuff happens:
|
What is the status here? |
I just encountered the issue that:
|
Warning:
The string it's breaking on is |
I was using an old reference. Now that I know to use http://proselint.com/checks/ I can see that the correct binding is |
Can anyone confirm if this was resolved? Nothing seems to reference it. |
I appear to be having this issue. {
"checks": {
"typography.symbols.curly_quotes": false,
"typography.symbols.sentence_spacing": false,
"typography.symbols.ellipsis": false,
"annotations.misc": false
}
} annotations and curly quotes is working but I'm still getting sentence spacing and ellipsis warnings despite my config file disabling them. "checks": {
"typography.symbols": false,
"annotations.misc": false
} This has no errors but I'd prefer to disable each error I don't need as I encounter it instead of blanket disabling all of them. |
Thank you for the report, @mohkale. Can you tell us what version of proselint you are using, and if applicable show us the output of |
I'm on |
And the output of |
{
"checks": {
"airlinese.misc": true,
"annotations.misc": false,
"archaism.misc": true,
"cliches.hell": true,
"cliches.misc": true,
"consistency.spacing": true,
"consistency.spelling": true,
"corporate_speak.misc": true,
"cursing.filth": true,
"cursing.nfl": false,
"cursing.nword": true,
"dates_times.am_pm": true,
"dates_times.dates": true,
"hedging.misc": true,
"hyperbole.misc": true,
"jargon.misc": true,
"lexical_illusions.misc": true,
"lgbtq.offensive_terms": true,
"lgbtq.terms": true,
"links.broken": false,
"malapropisms.misc": true,
"misc.apologizing": true,
"misc.back_formations": true,
"misc.bureaucratese": true,
"misc.but": true,
"misc.capitalization": true,
"misc.chatspeak": true,
"misc.commercialese": true,
"misc.composition": true,
"misc.currency": true,
"misc.debased": true,
"misc.false_plurals": true,
"misc.illogic": true,
"misc.inferior_superior": true,
"misc.institution_name": true,
"misc.latin": true,
"misc.many_a": true,
"misc.metaconcepts": true,
"misc.metadiscourse": true,
"misc.narcissism": true,
"misc.not_guilty": true,
"misc.phrasal_adjectives": true,
"misc.preferred_forms": true,
"misc.pretension": true,
"misc.professions": true,
"misc.punctuation": true,
"misc.scare_quotes": true,
"misc.suddenly": true,
"misc.tense_present": true,
"misc.waxed": true,
"misc.whence": true,
"mixed_metaphors.misc": true,
"mondegreens.misc": true,
"needless_variants.misc": true,
"nonwords.misc": true,
"oxymorons.misc": true,
"psychology.misc": true,
"redundancy.misc": true,
"redundancy.ras_syndrome": true,
"security.credit_card": true,
"security.password": true,
"sexism.misc": true,
"skunked_terms.misc": true,
"spelling.able_atable": true,
"spelling.able_ible": true,
"spelling.athletes": true,
"spelling.em_im_en_in": true,
"spelling.er_or": true,
"spelling.in_un": true,
"spelling.misc": true,
"terms.animal_adjectives": true,
"terms.denizen_labels": true,
"terms.eponymous_adjectives": true,
"terms.venery": true,
"typography.diacritical_marks": true,
"typography.exclamation": true,
"typography.symbols": true,
"typography.symbols.curly_quotes": false,
"typography.symbols.ellipsis": false,
"typography.symbols.sentence_spacing": false,
"uncomparables.misc": true,
"weasel_words.misc": true,
"weasel_words.very": true
},
"max_errors": 1000
} |
Ah, I apologize, I didn't see which checks you had specifically disabled before. That's related to another ongoing issue - the disabling of individual subchecks. |
Could you direct me to the issue ticket for it? This is the only issue I could find that seemed to be related to it. |
All related issues are attached to #1172 |
Closing in favour of #1172. |
Are there any guidelines on how to use
.proselintrc
?I've found a
.proselintrc
in/usr/lib/python2.7/site-packages/proselint
and can modify the checks there, but it doesn't seem like I can override it with something like~/.proselintrc
.Also, is there any way to disable a particular rule? I want to stop curly quote checking, but the following doesn't work in
.proselintrc
.Is there any way to keep the rest of the checks in
typography.symbols
, but disable one rule?The text was updated successfully, but these errors were encountered: