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

CWE: Optional tests #779

Merged
merged 11 commits into from
Sep 25, 2024
Merged

Conversation

tschmidtb51
Copy link
Contributor

- addresses parts of oasis-tcs#660, oasis-tcs#735
- add test to prevent usage of deprecated CWEs
- addresses parts of oasis-tcs#660, oasis-tcs#735
- add invalid examples for 6.2.23
- add valid examples for 6.2.23
- fix spelling mistakes
- addresses parts of oasis-tcs#660, oasis-tcs#737
- add optional test to suggest usage of latest corresponding version in CWE
- fix spelling mistake
- addresses parts of oasis-tcs#660, oasis-tcs#737
- add invalid examples for 6.2.24
- add valid examples for 6.2.24
- addresses parts of oasis-tcs#660, oasis-tcs#743
- add optional test to prevent vulnerability mapping to CWE that is not allowed
- addresses parts of oasis-tcs#660, oasis-tcs#743
- add invalid examples for 6.2.25
- add valid examples for 6.2.25
- addresses parts of oasis-tcs#660, oasis-tcs#743
- add optional test to prevent vulnerability mapping to CWE that is not allowed without review
- addresses parts of oasis-tcs#660, oasis-tcs#743
- add invalid examples for 6.2.26
- add valid examples for 6.2.26
@tschmidtb51 tschmidtb51 added editor-revision already worked on in the editor revision csaf 2.1 csaf 2.1 work labels Aug 27, 2024
@tschmidtb51 tschmidtb51 self-assigned this Aug 27, 2024
@tschmidtb51
Copy link
Contributor Author

@sthagen: There are 2 cite parts, where I quote the page given in the link directly. I guess the format is incorrect. Can you please also point out in your review how to do that correctly?

- addresses parts of oasis-tcs#779
- remove mentioning of testfiles that never existed in the first place
Copy link
Contributor

@sthagen sthagen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

csaf_2.1/prose/edit/src/tests-02-optional.md Outdated Show resolved Hide resolved
csaf_2.1/prose/edit/src/tests-02-optional.md Outdated Show resolved Hide resolved
- addresses review comments of oasis-tcs#779
- fix title case
@tschmidtb51
Copy link
Contributor Author

@sthagen - title case fixed :-)

@santosomar
Copy link
Contributor

- addresses parts of oasis-tcs#779
- correct links as suggested by @sthagen
Copy link
Contributor

@sthagen sthagen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@tschmidtb51 tschmidtb51 merged commit 5e70955 into oasis-tcs:editor-revision-2024-08-28 Sep 25, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
csaf 2.1 csaf 2.1 work editor-revision already worked on in the editor revision
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants