[GitHub] [geode] PurelyApplied commented on pull request #2598: GEODE-5855 hide jobs that are not gating in default pipeline view

["PurelyApplied (GitHub)" <gi...@apache.org>]

I almost hate to comment on one-line PRs, but if the intent is "only the things that act as blocking for `UpdatePassingRef`, then we should use the same as that block:

```
 - name: UpdatePassingRef	 - name: UpdatePassingRef
   public: true	   public: true
   serial: true	   serial: true
   plan:	   plan:
   - get: geode	   - get: geode
     passed:	     passed:
     {% for test in tests if not (test.name=="StressNew" or test.name.startswith("Windows")) -%}	     {% for test in tests if not (test.name=="StressNew" or test.name.startswith("Windows")) -%}
     - {{test.name}}TestOpenJDK8	     - {{test.name}}TestOpenJDK8
     {% endfor %}	     {% endfor %}
     trigger: true
```

and/or macro / explicitly define the "these are the jobs" list in our Jinja parameters somewhere.

[ Full content available at: https://github.com/apache/geode/pull/2598 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org