…ed fixes (#81)

* integrate mypy type checking into CI
* resolve #75
* resolve #73
* run pylint workflow on changes to workflow file

- fix a few long line warnings
- add rich to the py dev reqs
- add types-PyYAML to dev reqs

* resolve #76


normalize submodule path separators per runner's OS

* add unit tests for ignored paths

also make sure the current working directory is appended to the .gitmodules path

* run pytest in CI

- add pytest to dev reqs
- rename run-pylint.yml to run-dev-tests.yml
- install cpp-linter pkg in run-dev-tests.yml workflow
- upgrade actions/setup-python to v2 in run-dev-tests.yml workflow
- rename step in run-dev-tests.yml workflow

* resolve #74 and add unit tests for it

This allows the `lines-changed-only` option to accept the following values:
- false: All lines in a file are analyzed
- true: All lines in the diff are analyzed (including unchanged lines)
- strict: Only lines in the diff that contain additions are analyzed

gitignore some test resources as they are fetched running the tests.

Retain original repo structure when fetching files. This is only used when the user doesn't checkout the repo being analyzed.

* improve and test CLI arg parsing

* add unit test for database

also fix a typo in demo.hpp variable name

* switch to pre-commit; run pytest in a matrix

* parametrize  unit tests

* fix changes to dev-test yaml

* make database path absolute

* use pathlib to resolve() relative paths in test

* let db path be independent of repo-root w/o docker

* this fails locally

* show me value for RUNNER_WORKSPACE

* use gh workspace if not in docker env

* simplify the paths' concatenation

* fix assignment of `Globals.FILES`

* treat unsupport events like a push

* explicitly set a env var when using docker

* ammend some logic

* show me some contents of folders on docker

* hardcode the correct worspace dir

* remove unused import

* try detecting the docker env better

* set the docker env var w/ a str

* assume that RUNNER_WORKSPACE is abs

* use only relativer path to db on docker

* Revert "use only relativer path to db on docker"

This reverts commit 37e3083.

* better test coverage

* mkdocstring doesn't support `Tuple[x, x]` type

see mkdocstrings/griffe/#95

* fix workflow

* revert changes to `get_line_cnt_from_cols()`

* upload without codecov token??

* ensure LF used on demo src

* update workflow

* re-implement newer `get_line_cnt_from_cols()`

* switch to pyproject.toml (setup.py is a dummy now)

* resolve version exe path better

* ammend make_annotations() (& its tests)

* adjust for pypi releases

* allow running locally & fix duplicate log cmds

* use pathlib.Path to open files

* rely on pathlib, update pkg name & CI workflows

* `is_relative_to()` introduced in python v3.9

* `lstrip(".")` from file's extension

* tidy v13 use abs path to std libs in output error

* build/install wheel in test CI

* oops, use correct artifact name

* remove identifying info from test's event payload

* make files list simpler for any event

* ammend walking the repo files/folders

This could probably get changed to using pathlib's glob mechanism.

* replace os.walk() with pathlib's rglob()

* only upload coverage report once from test CI

* use OS dependent path separators for DB path

* resolve #82

* ensure tidy fixit_lines end w/ a LF

* workflow continue only if `latest` tag was added

* include v7-9 for CI tests

* fix docs about `parse_ignore_option()`

Also, docs show type annotations in the function signatures

* fix `TidyNotification.__repr__()`

* tell CI to download clang-tools v7-9

* support older versions of clang-tidy YML output

* don't try to traverse a `None` obj

* add common hooks to pre-commit config

* add more configs to toml

This removes the need for a separate .coveragerc file

pytest can be executed without args based on the config in the toml

* update event triggers on dev-test CI workflow

* sort requirements.txt contents

* split pre-commit step into its own CI workflow

update version of used (external) actions

* pleasing pre-commit hooks

* fix run-test.yml (trailing whitespaces)

* revise README & remove .ci-ignore file

* add code coverage badge to README

* Fix add tag failed when more than 1 PR open(#84)

The README states that the value of true focuses on
lines in diff with additions only.
Setting it to 'diff' will use the original behavior (meaning
entire diff chunks with unchanged lines).

However, I forgot to change this value's
interpretation in the src code (& test).

This commit makes the README accurate.