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

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

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

Conversation

yasuda-shin
Copy link
Contributor

概要

「AmebaLIFE事業本部」をtextlintのルールに追加してみました。
サービス用語ではない気がしたので別のルールとして切り出しつつ「Business」だと割と多様な言葉を囲い込めそうな気がしつつもっといい名称があれば大募集中です。

コーポレートも頭をよぎったのですがちょっと会社色強くなっちゃうかと思った次第です。

@yasuda-shin
Copy link
Contributor Author

yasuda-shin commented Oct 3, 2023

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

error  Ameba LIFE事業本部 => AmebaLIFE事業本部  prh

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

error  Ameba LIFE事業本部 => AmebaLIFE事業本部  prh

となったのでservice.ymlを合わせてもチェック範囲網羅できてそうでした。

@herablog
Copy link
Member

herablog commented Oct 4, 2023

もしくは、厳密な意味とずれちゃいますがAmebaLIFE関連は近い場所(今回はprh-service.yml)に記述しちゃうのはどうですかね?許容範囲内なので管理しやすさを優先してもいいかなというのがあります〜

サイトにどう掲載したいか確認して、カテゴリ分けたいであれば今のままでも良さそうです!

@yasuda-shin
Copy link
Contributor Author

54d3b03

☝️AmebaLIFEの付近に記載する形でservice側に転記してみました。
そこまで違和感ないですね。カテゴリ分け難しい。

@@ -61,6 +61,16 @@ rules:
- from: Ameba ライフ
to: AmebaLIFE

- 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 なるものがあるみたいなのですがうまくいかなさそうで。

ちょいお手上げです😇

@yasuda-shin yasuda-shin force-pushed the feature/business-text-rule branch from 54d3b03 to 6c79f82 Compare October 4, 2023 06:38
@yasuda-shin yasuda-shin changed the title ビジネス向け用語のルールを追加 AmebaLIFE事業本部をルールに追加 Oct 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants