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.

Sign up for GitHub

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

Jump to bottom

Merge #164 to master #168

Merged
merged 95 commits into from
Dec 16, 2024
Merged

Merge #164 to master #168

merged 95 commits into from
Dec 16, 2024

Conversation

JasperCraeghs
Copy link
Member

No description provided.

JasperCraeghs and others added 30 commits November 29, 2024 12:00
@JasperCraeghs
Fatal error when a mandatory key is missing from the config; no more …
61bfc61 
…print when a checker is not explicitly disabled in the config
@JasperCraeghs
address warning about incomplete config
b923ad3
@JokeWaumans
Use logging module instead of print
755044b
@JokeWaumans
Update tests to catch logs instead of prints
fdba6f4
@JokeWaumans
Fix line endings to LF
18b4502
@JokeWaumans
Order imports; Delete unused imports
63f9fbd
@JokeWaumans
Delete default mode 'r' when opening a file
9baf00d
@JokeWaumans
Use f-string
6598d8b
@JokeWaumans
Delete line end of file
04df6a5
@JokeWaumans
Reorder imports of scripts and init file
d83f085
@JokeWaumans
Delete newlines
cb47ff7
@JokeWaumans
Delete default usage of utf-8
ffcb69e
@JokeWaumans
Use f-string
a41079a
@JokeWaumans
Delete default mode 'r' when opening a file
fa12bf5
@JokeWaumans
Use f-string for --version
e14b35f
@JokeWaumans
Reduce the amount of prints by combining them
b5bdf27
@JokeWaumans
Update tests to match the reduced prints
8b75a73
@JokeWaumans
Fix test; assertNoLogs added in version 3.10
d304b96
@JokeWaumans
Use f-string
16e8dad
@JokeWaumans
Update design
d13853b
@JokeWaumans
Fix flake8
57ecd2f
@JokeWaumans
Add indentation
05b63af
@JokeWaumans
Fix setting logging level
a3e6d8d
@JasperCraeghs
Merge branch 'lf' of https://github.com/melexis/warnings-plugin into …
3bcf95a 
…logging
@JasperCraeghs
Merge branch 'master' of https://github.com/melexis/warnings-plugin i…
2299240 
…nto logging
@JokeWaumans
Add verbosity tests
7e91146
@JokeWaumans
Merge branch 'logging' of github.com:melexis/warnings-plugin into log…
5237cec 
…ging
@JokeWaumans
Capture all prints in other tests
cca6fc5
@JokeWaumans
Use run_test_with_logging in robot test
93864a7
@JokeWaumans
Use capitalized names of checkers
6e4e052
JasperCraeghs and others added 28 commits December 13, 2024 22:34
@JasperCraeghs
Use 1 logger per checker class; include failure msg for junit/robot c…
5573624 
…hecker in verbose mode
@JasperCraeghs
Debug level for --output mode only; logger of sub-checkers must not p…
e2e292d 
…ropagate to logger of parent checker
@JasperCraeghs
remove local debug file
cedb484
@JasperCraeghs
set level of LOGGER to simulate --verbose
a2e434b
@JasperCraeghs
remove unused import
8d41335
@JasperCraeghs
filter out non-debug logs from --output file
befb57f
@JasperCraeghs
fix bug that resulted in empty --output file when multiple sub-checke…
38a5ec3 
…rs used
@JasperCraeghs
prepare logger for capture with pytest fixture
6912465
@JasperCraeghs
update reference --output files: failure message is appended with pip…
6fa8d17 
…e char io colon
@JasperCraeghs
use capsys fixture to test the custom format of our loggers
b73f402
@JasperCraeghs
Merge branch 'urls' of https://github.com/melexis/warnings-plugin int…
d94412a 
…o logging
@JokeWaumans
Use double quotes; Adapt strings to line lenght = 120
6fde428
@JasperCraeghs
Remove unused kwarg
a3100df
@JasperCraeghs
support subclassing in name_repr
efbd2d3
@JokeWaumans
Add tests for testing --verbose and --output options for Coverity
1c0a1a8
@JokeWaumans
Add tests for testing --verbose and --output options for Polyspace
039a521
@JokeWaumans
Merge branch 'logging' of github.com:melexis/warnings-plugin into log…
9e83f10 
…ging
@JasperCraeghs
correct docstring
936141f
@JasperCraeghs
remove unused var 'padding'
6dc425d
@JasperCraeghs
remove 'subchecker' attr by relying on 'name' attr
ad66c25
@JasperCraeghs
Merge branch 'logging' of https://github.com/melexis/warnings-plugin …
ad3cdc7 
…into logging
@JasperCraeghs
fix logic and naming
d1a3c3a
@JokeWaumans
Add tests to test --output option of Robot
4293591
@JokeWaumans
Use shorter tsv file to test verbose option of Polyspace
384cb46
@JokeWaumans
Add Polyspace test with the use of exclude and verbose options together
a40dd4c
@JokeWaumans
Merge branch 'logging' of github.com:melexis/warnings-plugin into log…
6f1f32e 
…ging
@JokeWaumans
Update design
bd633de
@JasperCraeghs
Merge pull request #164 from melexis/logging
1d74c5e 
Draft: Output a cleaner and more compact summary
@JasperCraeghs JasperCraeghs marked this pull request as ready for review December 16, 2024 20:57
@JasperCraeghs JasperCraeghs merged commit fd05e60 into master Dec 16, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@JasperCraeghs @JokeWaumans