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

Убрать зависимость от go-openapi #80

Closed
lansfy opened this issue Mar 2, 2021 · 6 comments
Closed

Убрать зависимость от go-openapi #80

lansfy opened this issue Mar 2, 2021 · 6 comments

Comments

@lansfy
Copy link
Contributor

lansfy commented Mar 2, 2021

Как я понимаю go-openapi используется только в checker_scheme, который вообще по-умолчанию в библиотеке не используется. Однако эта зависимость есть в go.mod и go.sum.

Может убрать из последних файлов зависимость от go-openapi? Большинству пользователей библиотеки, как я понимаю это не нужно, а дефолтная зависимость тем не менее присутствует, что добавляет в проект не только зависимость от go-openapi, но и от пачки других библиотек.

@lansfy lansfy changed the title Зависимость от go-openapi Убрать зависимость от go-openapi Mar 2, 2021
@fetinin
Copy link
Contributor

fetinin commented Mar 3, 2021

Отпишу здесь для истории то же что и обсуждали раньше.

А действительно ли она так мешает? Кажется единственный вариант это вынести этот чекер в отдельный модуль, тогда его можно и нужно будет ставить отдельно. К тому же, этот чекер используется в main.go и это будет обратно несовместимое изменение.

@luza
Copy link
Contributor

luza commented Mar 3, 2021

Может, проголосуем и уберем ее, действительно? У этой фичи тёмная история, и я правда сомневаюсь, что про нее кто-нибудь вспомнит.

Я за убирание в основном потому, что она выпадает из концепции утилиты и размывает ее сферу применения.

@lansfy
Copy link
Contributor Author

lansfy commented Mar 3, 2021

Кажется единственный вариант это вынести этот чекер в отдельный модуль, тогда его можно и нужно будет ставить отдельно.

Не обязательно, можно оставить файл в дереве, там где он сейчас и находится, просто убрать поддержку этой фичи из command-line утилиты. Тем кто захочет использовать данный чекер в своем проекте может это сделать c помощью фичи которую я недавно добавил в #75

К тому же, этот чекер используется в main.go и это будет обратно несовместимое изменение.

Да, но об этой фиче никто нигде не писал, так что не понятно каким чудом потенциальный пользователь о ней бы узнал.
Да и я скептически отношусь к command-line утилите, потому что с ней нельзя протестировать сервисы с моками, что фактически вынуждает использовать gonkey как библиотеку (а без моего недавнего фикса использовать этот чекер в библиотеке просто не получилось бы).

@fetinin
Copy link
Contributor

fetinin commented Mar 9, 2021

Уточню на днях использует ли кто-то этот функционал в компании. Если этим действительно никто не пользуется, то я не против её выпилить.

@fetinin
Copy link
Contributor

fetinin commented Mar 10, 2021

Не нашел тех кто бы использовал эту фичу

@fetinin
Copy link
Contributor

fetinin commented Mar 19, 2021

🚀 Released in v1.4.0 🚀

@fetinin fetinin closed this as completed Mar 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants