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

Normalize unknown|undefined behaviour #340

Merged
merged 27 commits into from
Jan 22, 2024
Merged

Normalize unknown|undefined behaviour #340

merged 27 commits into from
Jan 22, 2024

Conversation

hesa
Copy link
Contributor

@hesa hesa commented Jan 16, 2024 •
edited
Loading

These commits addresses #336

changes:

  • problems (e.g unknown license in inbound or outbound) store instead of raising exceptions directly
    • if any problem raise exception and exit with error code
    • if optionally --ignore-problem is set, report problem but do not raise exception
  • jsconschema test validate result from verify_outbounds_inbound. The schema is used in tests/
  • jsconschema test validate result from verify. The schema is used in tests/

hesa added 22 commits January 17, 2024 00:26
@hesa
arbiter: Store dependencies problem
402bdd0 
use get to get default values if None
@hesa
compatibility: store problems (no exception)
b8b867c
@hesa
lic_com: store problems (no exception raised)
6996c05
@hesa
requirement: add jsonschema for test
49e38f6
@hesa
test: rewrite tests for no exception
a784fdf
@hesa
test: new test files for problems and verify
6ec24fa
@hesa
schema: add file for compat checks
1bd34f7
@hesa
reuse: add (c) and license
78986d3
@hesa
doc: add note on aliases via foss-licenses
68b9b37
@hesa
main: new option: --ignore-problems
3e61ffe 
command can provide exit code for main
@hesa
arbiter: store unique problems in one array
72931cf
@hesa
main: optionally allow problems
9cf44f4 
if problem => raise exception. Optionally no exception.
@hesa
validation: add schema for compat report
fa1306f
@hesa
validation: remove old schema
8fae480
@hesa
validation: add schema for verification report
4516c4f
@hesa
test: add mock variable ignore_problems
b186eff
@hesa
test: use renamed file
b2beef8
@hesa
test: add schema validation
09fa13e
@hesa
test: use renamed file
e4b6fa8
@hesa
reuse: add JSON schemas
b8bc85d
@hesa
merge: 'main' to hesa-normalize-unknown-behaviour
e8d584d
@hesa
test: fix typo
b960a2f
@hesa hesa merged commit fad7087 into vinland-technology:main Jan 22, 2024
5 checks passed
@hesa hesa mentioned this pull request Jan 25, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@priv-kweihmann priv-kweihmann priv-kweihmann left review comments

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
@hesa @priv-kweihmann