IJHack / QtPass / 23800989536
21%
master: 7%

LAST BUILD BRANCH: main
DEFAULT BRANCH: master
Ran
Jobs 1
Files 62
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 19.817% (+1.3%) from 18.508%
23800989536

push

github

web-flow 
style: use slots/signals instead of Q_SLOTS/Q_SIGNALS (#867)

* style: use slots/signals instead of Q_SLOTS/Q_SIGNALS

Follow Qt's recommended convention for application code.

Both forms are equivalent in Qt5/6, but Qt's own documentation and
conventions prefer the simpler 'slots:' and 'signals:' forms for
application code. The Q_* macros are defensive alternatives for
library headers that must compile in hostile environments where
'slots' or 'signals' might be defined as macros.

* 📝 Add docstrings to `fix/slots-signals-style`

Docstrings generation was requested by @annejan.

The following files were modified:

* `src/configdialog.h`
* `src/deselectabletreeview.h`
* `src/executor.h`
* `src/keygendialog.h`
* `src/mainwindow.h`
* `src/passworddialog.h`
* `src/qprogressindicator.h`
* `src/qpushbuttonasqrcode.h`
* `src/qpushbuttonshowpassword.h`
* `src/qpushbuttonwithclipboard.h`
* `src/qtpass.h`
* `src/singleapplication.h`
* `src/trayicon.h`

These files were kept as they were:
* `src/pass.h`
* `src/usersdialog.h`

* 📝 CodeRabbit Chat: Add unit tests for PR changes

* Update tests/auto/ui/tst_ui.cpp

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>

---------

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: Anne Jan Brouwer <brouwer@annejan.com>

999 of 5041 relevant lines covered (19.82%)

7.77 hits per line

Jobs
ID Job ID Ran Files Coverage
1 23800989536.1 62
19.82
 GitHub Action Run
Source Files on build 23800989536