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

RedPen の InvalidSymbol ルールに例外を設定したい #17

Open
5t111111 opened this issue Mar 10, 2018 · 5 comments
Open

RedPen の InvalidSymbol ルールに例外を設定したい #17

5t111111 opened this issue Mar 10, 2018 · 5 comments

Comments

@5t111111
Copy link
Member

実際に困ってるのは (, ) 半角括弧を InvalidSymbol が許容してくれないので、文中に foo(arg) みたいな記述をしたいとき引っかかるってやつ。

シンボル設定をしてしまうと多分それはちょっと違うので、例外を設定したりすることはできないだろうか??

@makenowjust
Copy link
Member

そもそもインラインコードに対してRedPenの処理が走るのがおかしい気がするんですけどどうなんでしょうかね‥‥。

@5t111111
Copy link
Member Author

わからんですね…

@5t111111
Copy link
Member Author

これ、ちょっとどうするのがいいのかわかんなかったので、とりあえず自分のところで出た分は (健全ではないですが) 該当の節で InvalidSymbol を抑制することにしています。

https://github.com/crystal-jp/techbookfest4/pull/18/files#diff-b8382241647f0385252faae39d98ef6eR37

あとは InvalidSymbol を warning 扱いにするとかしか思いつかなかったので…

@makenowjust
Copy link
Member

ちょうどこれを改善するようなPull RequestがRedPenに存在するっぽいですね

redpen-cc/redpen#819

@5t111111
Copy link
Member Author

あ、そうだった。そう、そこも見て一旦諦めちゃったんですよね。

一応抑制してる節ではそれ以外のエラーは出てないのは確認した上で抑制したので、ひとまずこのままにしておきます。

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

No branches or pull requests

2 participants