greenwave

#188 The remote rule does not need a test_case_name of its own.
Merged by ralph. Opened by ralph.
no-testcase-name  into  master

Download 188.patch

The test case names get defined by further rules in dist-git's gating.yaml file.

:+1: but I was planning to submit a PR to refactor this to not be a rule at all, but rather just an extra piece of logic in the policy loading part of the POST /decision API.

I was staring at this for a while when I was working on PR#184 and I don't think it makes sense as a rule in a policy.

The PR looks good.

👍 but I was planning to submit a PR to refactor this to not be a rule at all, but rather just an extra piece of logic in the policy loading part of the POST /decision API.
I was staring at this for a while when I was working on PR#184 and I don't think it makes sense as a rule in a policy.

What were you thinking about?

Thanks both. Let's take up @dcallagh's aside elsewhere. Thanks for the heads up.

Pull-Request has been merged by ralph
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.