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.

Sign up for GitHub

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

AmebaLIFE事業本部をルールに追加 #59

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

AmebaLIFE事業本部をルールに追加 #59

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
0df47da
feat: add business text rule
yasuda-shin Oct 3, 2023
dfdb053
fix: ameba life division text rule
yasuda-shin Oct 4, 2023
6c79f82
feat: add ameba life division text rule
yasuda-shin Oct 4, 2023
2f26be5
fix: change ameba life devision position
yasuda-shin Oct 4, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 22 additions & 0 deletions lib/prh-service.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,28 @@ rules:
- from: AMEBLO
to: Amebaブログ

- expected: AmebaLIFE事業本部
pattern:
Copy link
Member

Choose a reason for hiding this comment

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

ちなみにすごい厳密にいうと 「Amebaライフ事業本部」とかは校正できなそうですけど、許容(例外)にします？厳密にする場合「事業本部」があるかどうかで判別しないとかもです。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

6c79f82
確かに。おとなしくAmeba LIFEに合わせて事業本部の方もパターン揃えてみました☝️

Copy link
Member

Choose a reason for hiding this comment

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

こりだと AmebaLIFE事業本部 が AmebaLIFE => Ameba LIFE prh で補正されちゃう気がしますがどうでしょう？prhファイルだと統合テストやりにくい(できますが)ので、example/ameba.md でやっておくといいかもです。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

2f26be5

ロジックで解決できず苦しいところはあるんですが AmebaLIFE事業部 を Ameba LIFE より前に記載することで回避してみたんですがどうでしょうかw

Copy link
Member

Choose a reason for hiding this comment

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

あり〜変わってないのわいだけですかね？ example.mdをこんな感じにしてます

AmebaLIFE事業本部を利用するには、アメーバIDの取得が必要です。サイバーエージェントが運営するサービスは他にも、AmebaNews、ABEMA、Owndなどがあります。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

うーん。ダメですかねー。

https://github.com/textlint-rule/textlint-rule-prh/blob/master/test/fixtures/example-prh.yml#L67-L79
なんか調べてたのですがtextlintに単語境界？の処理を提供するオプション wordBoundary なるものがあるみたいなのですがうまくいかなさそうで。

ちょいお手上げです😇

- Ameba LIFE事業本部
- AmebaLife事業本部
- Ameba Life事業本部
- Ameba life事業本部
- Amebaライフ事業本部
- Ameba ライフ事業本部
specs:
- from: Ameba LIFE事業本部
to: AmebaLIFE事業本部
- from: AmebaLife事業本部
to: AmebaLIFE事業本部
- from: Ameba Life事業本部
to: AmebaLIFE事業本部
- from: Ameba life事業本部
to: AmebaLIFE事業本部
- from: Amebaライフ事業本部
to: AmebaLIFE事業本部
- from: Ameba ライフ事業本部
to: AmebaLIFE事業本部

- expected: Ameba LIFE
pattern:
- AmebaLIFE
Expand Down
Loading