search

minsk-hackerspace / hackerspace.by

Website of Minsk Hackerspace (Ruby on Rails)
https://hackerspace.by/
11 stars 20 forks source link

Revert rogue change slipped without reviews in #511 #514

Closed abitrolly closed 3 years ago

abitrolly commented 3 years ago

It is not related to "versions bumped accordint to security warnings"

This partially reverts commit 6578f66c5b431159dac3ced1f48296ebce2f5f75.

abitrolly commented 3 years ago

@k2m30 care to explain why you want this change to stay there unreverted after you've injected it in #511?

k2m30 commented 3 years ago

Толик, не выделывайся. Хочешь сделать правильно, раздели и сделай. Как CI сломать в одно лицо, поставить web hook неработающий и свалить, так ты молодец. А потом как security issues разгребать, потому что ты сломал security auto-merge с неработающим CI, чтобы сделать минорную правку, так в кусты.

Код давно на сайте, если хочешь потрахаться с гитом и разделить – на здоровье.

abitrolly commented 3 years ago

Так, я вижу персональные наезды вместо объяснений. Про то, кто сломал "CI в одно лицо" можешь почитать на https://blog.travis-ci.com/2021-05-07-orgshutdown Когда дочитаешь, можешь извиниться за свой наезд, и проедем.

По поводу правки, я не буду поддерживать твою попытку свалить от ответа. Если не захочешь объяснить сам, я придумаю что-нибудь за тебя.

Код давно на сайте, если хочешь потрахаться с гитом и разделить – на здоровье.

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

Хочешь сделать правильно, раздели и сделай.

Разделил и откатываю.

k2m30 commented 3 years ago

Ты добавил гитлабовский вебхук, который сломал всё (потому что open source). и мне вместо того, чтобы добавить две строчки на сайт пришлось трахаться ночью с зависимостями и чинить это всё.