search

LaravelRUS / laravel-karma-bot

Laravel\Karama bot service with Gitter and Slack integration
http://karma.laravel.su/
MIT License
38 stars 11 forks source link

fixed syntax error #85

Closed symbios-zi closed 8 years ago

symbios-zi commented 8 years ago

Ошибка в слове larvel => laravel

SerafimArts commented 8 years ago

Всё ок, только надо вообще над этой регуляркой подумать. @jhaoda вне себя от упоминания кейворда "доки". Есть идеи?

ghost commented 8 years ago

Думаю, достаточно оставить несколько основных словосочетаний (также как сделано в "погугли"), к которым просто со временем все привыкнут.

(читай|ищи|смотри|см) (доки|доку|документацию) (по|про)

Т.е. для сработки должно присутствовать три слова подряд, по одному из каждой группы вариантов. Внеплановых сработок по идее сразу станет меньше. А те кто любит посылать в доку выберут себе наиболее привычное сочетание (из 24 вариантов комбинаций :)).

SerafimArts commented 8 years ago

Мне нравится :+1: