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

Few questions regarding the project #469

Closed
aol-nnov opened this issue Sep 1, 2024 · 11 comments
Closed

Few questions regarding the project #469

aol-nnov opened this issue Sep 1, 2024 · 11 comments

Comments

@aol-nnov
Copy link
Contributor

aol-nnov commented Sep 1, 2024

Hey, @fralx !

Let me ask you some questions about the project as I'm planing to make few contributions to it.

As for me, I'm a long-term cutereport user, been stuck to it till the end, ha! Time showed quite clear that cute report has been left unmaintained long time ago, so, I finally decided to find an alternative and luckily stumbled upon LimeReport!

It appeared quite familiar to me (he-he) and, what was the most important - alive - qt6 support and such...

First of all, I'd like to contribute Debian packaging for LimeReport, so the first question is: what's the primary build system of the project as I've recently discovered that cmake and qmake builds are being installed into different locations, which is a bit confusing.

Also, I'd like to know if you continue to invest into qt5 as it failed to compile against 5.15.8 on Debian 12. Changes were minimal and trivial, but yet...

What is the accepted code style? While trying to guess, I've tried formatting with a few clang-format presets, but all of them introduce tremendous changes to the codebase. If there is no such an accepted style, I'd propose to define one and store it along the codebase to make new contributions more consistent with the codebase regardless the IDE used by the contributor.

Plus any other bits and pieces that coud come to my mind while diving into the project. Anyway, would be glad to hear from you regarding the above.

Best,
Andrey

P.S.: also, what's the primary language of this tracker? I'm okay with English, but we could switch Russian any time, if you'd like.

@fralx
Copy link
Owner

fralx commented Sep 2, 2024

Приветствую @aol-nnov ! Думаю будет проще на русском :)

I'd like to know if you continue to invest into qt5 as it failed to compile against 5.15.8 on Debian 12

Гляну, на данный момент, я не особо много времени уделяю проекту, поэтому могу пропускать какие-то изменения в Qt, которые приводят к поломкам сборки.

What is the accepted code style?

Стиль как говорится авторский :) Наверно ни один линтер так не отформатирует.
С идеей отформатировать код под один стандарт в целом согласен :) .

what's the primary build system of the project

Изначально использовался qmake, позже @Jihadist прикрутил cmake, qmake собирает более полный вариант. По cmake у меня руки не доходят его доделать, хотя в этом и есть смысл.

@aol-nnov
Copy link
Contributor Author

aol-nnov commented Sep 2, 2024

Спасибо, что ответил, @fralx !

Стиль как говорится авторский

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

По cmake у меня руки не доходят его доделать, хотя в этом и есть смысл.

я бы взялся перевести проект на cmake целиком. подумаю, как бы можно было ублажить тех, кто пользуется qmake... может, через pkg-config? (а то, так и вообще отказаться от qmake, как от отмирающей технологии)

кроме прочего, на днях придумал, как генерировать версию при скачивании архива с гитхаба так же, при сборке через гит. скоро залью pr.

две недели отпуска не должны пройти даром, хехе.

@fralx
Copy link
Owner

fralx commented Sep 3, 2024

я бы предложил сочинить что-то из clang-format

Можно попробовать

@aol-nnov
Copy link
Contributor Author

@fralx не хочу звучать агрессивно/провоцирующе/или еще как-то не уютно, но, глядя на отзывчивость реакции в проекте, кажется, нам дальше не по пути. Так бывает - нет времени/желания/еще чего-то.. Спасибо за интересный и полезный проект.

Я сделал несколько пуллл-реквестов #470 #471 и #472.

Больше не потревожу - продолжу отдельно на базе этих изменений.
Удачи! :)

@fralx
Copy link
Owner

fralx commented Sep 17, 2024

@aol-nnov ! Я очень ценю вклад всех участников проекта.
На самом деле, реально, не было времени, завал на работе :(.
Жаль, что так вышло :(.
Я не отказываюсь от предложенных изменений, просто, мне нужно некоторое время и некоторый ментальный ресурс.

@aol-nnov
Copy link
Contributor Author

No hard feelings, как говорится! :)

просто, мне нужно некоторое время и некоторый ментальный ресурс.

Прекрасно понимаю это. потому и написал - чтобы висящие парки не создавали избыточного давления, так сказать. Никаких гарантий, что меня завтра не завалит и все мои громкие обещания отложатся на полгода, скажем )))

Быть может, свидимся смёржимся еще! :-D

@fralx
Copy link
Owner

fralx commented Sep 17, 2024

Запустил проверки если пройдут смержусь с предложенным :)

@fralx
Copy link
Owner

fralx commented Sep 17, 2024

@aol-nnov ! Если желание не совсем пропало :), то там проверки в паре реквестов не прошли

@aol-nnov
Copy link
Contributor Author

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

впрочем, одним из пунктов моего дальнейшего путешествия значится распутывание всех загадок с хедерами и натравливание include what you use. но это потом )

@fralx
Copy link
Owner

fralx commented Sep 17, 2024

@aol-nnov Могу вечером глянуть :) А то от настроек кластера уже мозг кипит, может развеюсь

@aol-nnov
Copy link
Contributor Author

хорошо, как будет время )

там еще ребейзить надо руками, оказывается.. ну, это я сейчас подвину. понадеялся на появление волшебной кнопки rebase, как в гитлабе.

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