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

evaluator-rules: Pull in all missing rules from ORT's example rules #53

Merged
merged 3 commits into from
Sep 19, 2022

Conversation

fviernau
Copy link
Member

@fviernau fviernau commented Sep 15, 2022

The evaluator rules in this repository have been created based on ORT's example policy rules, meanwhile they deviated, but are quite redundant. Reduce the deviation use-case-wise by copying all rules from ORT's examples which are missing in this repository using ORT revision [1].

This is the first step towards making the 'ort-config' repository the single dedicated place to examplify policy rules. In the following ORT's example rules will be deleted, or rather minimized and turned into functional test assets, see also [2].

Note: The copied rules have been created as part of [3].

[1] 63e002ba57e7d49c96017fac2ff679de8a5b76df
[2] oss-review-toolkit/ort#5701
[3] oss-review-toolkit/ort#5621

Part of: oss-review-toolkit/ort#5621.

@fviernau fviernau requested a review from a team as a code owner September 15, 2022 09:42
@fviernau fviernau requested a review from tsteenbe September 15, 2022 09:42
@fviernau fviernau force-pushed the pull-in-rules-missing-from-ort-examples branch 2 times, most recently from fb7f7e4 to 6411fa7 Compare September 16, 2022 10:57
@fviernau fviernau marked this pull request as draft September 16, 2022 13:58
tsteenbe
tsteenbe previously approved these changes Sep 16, 2022
The evaluator rules in this repository have been created based on ORT's
example policy rules, meanwhile they deviated, but are quite redundant.
Reduce the deviation use-case-wise by copying all rules from ORT's
examples which are missing in this repository using ORT revision [1].

This is the first step towards making the 'ort-config' repository the
single dedicated place to examplify policy rules. In the following ORT's
example rules will be deleted, or rather minimized and turned into
functional test assets, see also [2].

Note: The copied rules have been created as part of [3].

[1] 63e002ba57e7d49c96017fac2ff679de8a5b76df
[2] oss-review-toolkit/ort#5701
[3] oss-review-toolkit/ort#5621

Signed-off-by: Frank Viernau <frank_viernau@epam.com>
Signed-off-by: Frank Viernau <frank_viernau@epam.com>
The rule `WRONG_LICENSE_IN_LICENSE_FILE_RULE` has been copied from ORT
in a previous change. As that rule is part of the "prior to open
sourcing" use case, it makes sense to use the list of approved licenses
for the check.

Signed-off-by: Frank Viernau <frank_viernau@epam.com>
@fviernau fviernau force-pushed the pull-in-rules-missing-from-ort-examples branch from 6411fa7 to 6ed2526 Compare September 19, 2022 07:17
@fviernau fviernau marked this pull request as ready for review September 19, 2022 07:18
@fviernau fviernau requested a review from tsteenbe September 19, 2022 07:18
@tsteenbe tsteenbe merged commit 6e2c64f into main Sep 19, 2022
@tsteenbe tsteenbe deleted the pull-in-rules-missing-from-ort-examples branch September 19, 2022 09:22
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