Skip to content

feat: Add Notice documentation header check.#1422

Merged
bdferris-v2 merged 3 commits intomasterfrom
issue/1324/rules_documentation_test
May 1, 2023
Merged

feat: Add Notice documentation header check.#1422
bdferris-v2 merged 3 commits intomasterfrom
issue/1324/rules_documentation_test

Conversation

@bdferris-v2
Copy link
Copy Markdown
Collaborator

As part of automated Notice documentation extraction (#1324), this PR adds a unit-test to enforce that each Notice has a Javadoc documentation header.

@bdferris-v2 bdferris-v2 requested a review from davidgamez May 1, 2023 05:09
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented May 1, 2023

✅ Rule acceptance tests passed.
New Errors: 0 out of 1426 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
Dropped Errors: 0 out of 1426 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
New Warnings: 0 out of 1426 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
Dropped Warnings: 0 out of 1426 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
0 out of 1426 sources (~0 %) are corrupted.
Commit: 805e74f
Download the full acceptance test report here (report will disappear after 90 days).
✅ Rule acceptance tests passed.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented May 1, 2023

✅ Rule acceptance tests passed.
New Errors: 0 out of 1426 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
Dropped Errors: 0 out of 1426 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
New Warnings: 0 out of 1426 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
Dropped Warnings: 0 out of 1426 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
0 out of 1426 sources (~0 %) are corrupted.
Commit: dcc5e2b
Download the full acceptance test report here (report will disappear after 90 days).
✅ Rule acceptance tests passed.

@bdferris-v2 bdferris-v2 merged commit 0073047 into master May 1, 2023
@bdferris-v2 bdferris-v2 deleted the issue/1324/rules_documentation_test branch May 1, 2023 17:27
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.

2 participants