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

Template dir option #1

Open
wants to merge 62 commits into
base: master
Choose a base branch
from
Open

Template dir option #1

wants to merge 62 commits into from

Conversation

frankwiles
Copy link
Member

No description provided.

frankwiles and others added 30 commits March 2, 2023 10:03
Pass options down to `templates()` everywhere it's called.
Change which jinja2 loader is used depending on existence of the
argument.
* Add thread to log messages if multithreading is active.
- Add comments and to collapse code and remove not needed endless loop.
The loop is always executed only once because because the first statement in the block isn't changed in a second iteration.
- Collapse also the data in the root.
- Use `pop(key)` instead of copying the whole dict ignoring the value.
- Fix evaluating of common path in nested HTML report.
The root_filter wasn't applied to the root directory in HTML report since there is no trailing path separator at the end. Now the directories always have a trailing path separator which is removed in the report generation. To get the common path part the root directory is excluded.
- Replace recursive generation of directory stats with a loop.
- Use a object instead of a dataclass and remove the recursive function and the tests for it.
- Improve detection of root directory.
* Move formats to sub directories.
* Move function print_reports from __main__.py to writer/__init__.py.
* Write global logger uppercase because it's constant.
* Move gcov data handling to a own format handler
* Use constants for the exit code.
* Add prefix of format to each option.
* Only forward the options of the format and some generic options to the handler.
* Add one config key for each long option.
Pass options down to `templates()` everywhere it's called.
Change which jinja2 loader is used depending on existence of the
argument.
- Refactor based on suggestion
- Refactors code to use new `gcovr/formats/` layout
- Renames option to `--html-template-dir`
- Fixes merge artifacts I created on accident
This allows the user to only have to override the certain templates
they wish to adjust and it will fall through to using the included
ones for any template names the user has not overriden in
`--html-template-dir`
frankwiles and others added 30 commits April 26, 2023 14:32
* Treat special case of an absolute path to source file.
* Rename options and reorder to have fnames first. Add typing informations.
Add gcc-12 and gcc-13 within docker.
Add sessions to run all gcc or all clang versions.
If the user stylesheet contains `/* Comment.Preproc */` the pygments CSS
is not added at the end.
* Add VSCode workspace and tasks.
* Add documentation for devcontainer.
* Add test for writable source directory.
* Continue on GCOV write error.
* Use 1.0 instead of 0.0 if `total` is 0
Co-authored-by: Michael Förderer <michael.foerderer@gmx.de>
Co-authored-by: Michael Förderer <michael.foerderer@gmx.de>
If a const and a choice is defined for an option the validation of the choice was skipped and the value was converted to a boolean.
Add additional themes `github.blue`, `github.green`, `github.dark-blue` and `github.dark-green`.

---------

Co-authored-by: Spacetown <michael.foerderer@gmx.de>
Co-authored-by: Michael Förderer <michael.foerderer@gmx.de>
* Add option --fail-under-decision and update exitcodes
Check length of lines dictionary before generating a range of key values

---------

Co-authored-by: Michael Förderer <michael.foerderer@gmx.de>
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

Successfully merging this pull request may close these issues.

5 participants