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

Issues/fix import path for default version sets, output of unittest outputter and null checks, deprecates support for python 3.9. #107

Conversation

varunmittal91
Copy link
Collaborator

No description provided.

Fixed null check expecting literal 'NULL' instead of a null value.
Fixed unittest outputter to not ignore tests in new format of demension followed by generic test case name.
Added new test case to ensure all check configs load without erroring out.

Signed-off-by: Varun Mittal <[email protected]>
@varunmittal91 varunmittal91 force-pushed the issues/fix-import-path-for-default-version-sets branch from c61a709 to 41865fd Compare December 18, 2023 13:16
@varunmittal91 varunmittal91 changed the title Issues/fix import path for default version sets and deprecates support for python 3.8. Issues/fix import path for default version sets. Dec 18, 2023
@varunmittal91 varunmittal91 changed the title Issues/fix import path for default version sets. Issues/fix import path for default version sets, output of unittest outputter and null checks. Dec 18, 2023
Signed-off-by: Varun Mittal <[email protected]>
Signed-off-by: Varun Mittal <[email protected]>
Signed-off-by: Varun Mittal <[email protected]>
@varunmittal91 varunmittal91 changed the title Issues/fix import path for default version sets, output of unittest outputter and null checks. Issues/fix import path for default version sets, output of unittest outputter and null checks, deprecates support for python 3.9. Dec 18, 2023
@varunmittal91 varunmittal91 force-pushed the issues/fix-import-path-for-default-version-sets branch from c6d9f1c to 6b8759f Compare December 18, 2023 17:32
@varunmittal91 varunmittal91 merged commit d211632 into finopsfoundation:dev Dec 19, 2023
15 checks passed
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.

1 participant