Skip to content
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

no-file-name-articles: forbid "teh" #100

Closed
wants to merge 1 commit into from

Conversation

ChALkeR
Copy link
Contributor

@ChALkeR ChALkeR commented Sep 26, 2016

It is sometimes used as an alternative to "the", so no-file-name-articles rule should probably also forbid it when forbidding "the".

Wiki reference: https://en.wikipedia.org/wiki/Teh.

😉

It is sometimes used as an alternative to "the", so
no-file-name-articles rule should also forbid it.
@wooorm
Copy link
Member

wooorm commented Sep 26, 2016

Awesome, I’m 👍 with that.

Could you add a test though? They’re embedded in each rule, so I’m looking for something like this.

@wooorm wooorm added the 🦋 type/enhancement This is great to have label Sep 26, 2016
@wooorm
Copy link
Member

wooorm commented Sep 26, 2016

@ChALkeR Oh and thanks for your first PR to a remark-related project ❤️

@wooorm wooorm closed this in e4640fe Oct 2, 2016
@wooorm wooorm added ⛵️ status/released 🗄 area/interface This affects the public interface 🧒 semver/minor This is backwards-compatible change labels Aug 15, 2019
@wooorm wooorm added the 💪 phase/solved Post is done label Aug 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🗄 area/interface This affects the public interface 💪 phase/solved Post is done 🧒 semver/minor This is backwards-compatible change 🦋 type/enhancement This is great to have
Development

Successfully merging this pull request may close these issues.

2 participants