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

fix: QA categories were being used only by the CLI engine #1156

Merged

Conversation

Akash-Nayak
Copy link
Contributor

@Akash-Nayak Akash-Nayak commented Mar 15, 2024

Updated the QA http rest engine of Move2Kube required for adding QA category enable/disable support to Move2Kube API #1155, #1151 (comment). For the disabled categories, the QA engine should automatically consider the default answer as the input answer.

Signed-off-by: Akash Nayak [email protected]

Copy link

Thanks for making a pull request! 😃
One of the maintainers will review and advise on the next steps.

@github-actions github-actions bot added the feat label Mar 15, 2024
Copy link

codecov bot commented Mar 15, 2024

Codecov Report

Attention: Patch coverage is 33.33333% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 14.89%. Comparing base (c8164a0) to head (c0ef8d7).

Files Patch % Lines
qaengine/engine.go 33.33% 4 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1156      +/-   ##
==========================================
+ Coverage   14.86%   14.89%   +0.02%     
==========================================
  Files          90       90              
  Lines        8376     8380       +4     
==========================================
+ Hits         1245     1248       +3     
+ Misses       6810     6809       -1     
- Partials      321      323       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Co-authored-by: Harikrishnan Balagopal <[email protected]>
Signed-off-by: Akash Nayak <[email protected]>
@HarikrishnanBalagopal HarikrishnanBalagopal changed the title feat: Update QA rest engine to support enabling and disabling QA categories when using the API fix: QA categories were being used only by the CLI engine Mar 20, 2024
@github-actions github-actions bot added fix and removed feat labels Mar 20, 2024
Copy link
Contributor

@HarikrishnanBalagopal HarikrishnanBalagopal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@HarikrishnanBalagopal HarikrishnanBalagopal merged commit fd897df into konveyor:main Mar 20, 2024
4 checks passed
@HarikrishnanBalagopal
Copy link
Contributor

Related #1154

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants