-
Notifications
You must be signed in to change notification settings - Fork 364
fix: Guard test activation for CI testing #724
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
Conversation
Signed-off-by: Anurag Dixit <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code conforms to Python style guidelines
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code conforms to C++ style guidelines
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code conforms to Python style guidelines
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code conforms to C++ style guidelines
@andi4191 can you put together an index of tests we disable for CI? We should be updating this list as we enable and disable tests |
@narendasan : I added the list of tests for tracking in this PR: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: Anurag Dixit [email protected]
Description
Aligning the CI testing guard for test_activation unit test case. This will make sure that the test suite is consistent across different release guarded by CI.
Fixes # (issue)
Type of change
Non-Breaking change
Please delete options that are not relevant and/or add your own.
Checklist: