fralx / LimeReport

Report generator for Qt Framework
http://limereport.ru/
Other
401 stars 152 forks source link

Feature/data roles #452

Closed DmitryZagorodnev closed 3 months ago

fralx commented 8 months ago

Приветствую ! Очень ценю Ваше желание внести свой вклад в развитие проекта.
У меня большая просьба, если Вы хотите, чтобы Ваш pull request был принят, верните форматирование кода к изначальному виду. Во-первых, форматирование кода для разработчика дело интимное :), а во-вторых, так проще смотреть какие изменения были реально сделаны. Ну и желательно это объединить в один commit.

DmitryZagorodnev commented 3 months ago

Добрый день. Извиняюсь за глупый вопрос - я не нашел в проекте файла с правилами форматирования кода (например .clang-format и т.п.) как я могу привести исходники в привычный для Вас вид?

Jihadist commented 3 months ago

@DmitryZagorodnev у вас в некоторых файлах есть изменения форматирования, думаю стоит их откатить и попытаться учитывать общее форматирование насколько это возможно. .clang-format можно попробовать принести отдельным pr сразу с проверкой в actions :)

DmitryZagorodnev commented 3 months ago

Добрый день. Я восстановил форматирование и объединил доработки в один коммит.

Jihadist commented 3 months ago

@fralx посмотри пожалуйста