Vadim Peretokin

Comments doubled up

1

Comments from the bot are starting to come through multiple times:  https://github.com/Mudlet/Mudlet/pull/6056#issuecomment-1097684740

Invalid suggestion created

10

Check out https://github.com/Mudlet/Mudlet/pull/5981#discussion_r814601177 - the action generated a syntactically invalid C++ suggestion by the looks of it?

review.py crash on invalid Qt path

2

My workflow installs Qt to `Qt5_DIR` (standard variable) which is `/home/runner/work/Mudlet/Mudlet/Qt/5.14.1/gcc_64`. That works fine for compiling purposes, but the clang-tidy-review action then tries to access an invalid path: ``` File...

It's common to have a `.clang-tidy` file at the root of the repo ([hundred of repos on github](https://sourcegraph.com/search?q=context:global+file:%5C.clang-tidy&patternType=literal)), what about supporting it out of the box so the file is...

Support clang-tidy-13

8

It would great to have support for the latest version of `clang-tidy` :+1:

Can't install on Ubuntu 19.10

1

Installing the requirements doesn't work with the following: ``` $ ./install-requirements-debian [sudo] password for vadi: Reading package lists... Done Building dependency tree Reading state information... Done git is already the...

Hi Xavier! Thanks a lot for making this library. Would you be able to put up some benchmarks on how it compares to string concatenation and the simple string.format()? That...

It's a great benchmark, single-header and easy to plug into your project.

Dual-screen support?

7

When using an external monitor as the main one, I completely miss breaks because the notification is limited to one monitor: 

Infrastructure: fix windows setup script to work with ccache

1

#### Brief overview of PR changes/additions Add ccache path to windows setup script #### Motivation for adding to Mudlet Fix Windows setup script #### Other info (issues closed, discussion etc)...