-
Notifications
You must be signed in to change notification settings - Fork 175
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
feat: Capture blacklisted specs inside archive #3664
feat: Capture blacklisted specs inside archive #3664
Conversation
a4cafff
to
2f56b4e
Compare
2f56b4e
to
9592ab3
Compare
@subpop When you get a chance if you could review the client code changes in this PR please. |
9592ab3
to
6e9af3a
Compare
From what I can see, this doesn't appear to alter the client execution of the collection routine, so this should be fine. |
6e9af3a
to
79d4f77
Compare
Awesome thank you for reviewing it. |
@kgrant-rh @jsvob Since y'alls team was the one who brought up the issue about not being able to tell if spec was blacklisted or not. When y'all have time if you could please review this PR, and let me know if these changes will help with that. |
* Capture the name of blacklisted specs, and dump them to a file in the archive. * Added parser and spec to parse the blacklisted specs and use in rules. Signed-off-by: Ryan Blakley <[email protected]>
79d4f77
to
eac8230
Compare
@ryan-blakley Yes, LGTM, thank you. It can help with resolving #3527 eventually. |
* Capture the name of blacklisted specs, and dump them to a file in the archive. * Added parser and spec to parse the blacklisted specs and use in rules. Signed-off-by: Ryan Blakley <[email protected]> (cherry picked from commit da20d33)
* Capture the name of blacklisted specs, and dump them to a file in the archive. * Added parser and spec to parse the blacklisted specs and use in rules. Signed-off-by: Ryan Blakley <[email protected]>
Check all that apply:
Complete Description of Additions/Changes:
Capture the name of blacklisted specs, and dump them to a file in the archive. Added parser and spec to parse the blacklisted specs and use in rules.