-
Notifications
You must be signed in to change notification settings - Fork 24
feat(KFLUXUI-267): replace GitHub with Git #72
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
/ok-to-test |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #72 +/- ##
==========================================
+ Coverage 80.01% 80.15% +0.13%
==========================================
Files 569 569
Lines 21380 21391 +11
Branches 5036 5041 +5
==========================================
+ Hits 17108 17146 +38
+ Misses 4248 4221 -27
Partials 24 24
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
/ok-to-test |
@@ -34,7 +34,7 @@ export class IntegrationTestsTabPage { | |||
) { | |||
this.verifySaveChangesIsDisabled(); | |||
UIhelper.inputValueInTextBoxByLabelName('Integration test name', integrationTestName); | |||
UIhelper.inputValueInTextBoxByLabelName('GitHub URL', githubURL); | |||
UIhelper.inputValueInTextBoxByLabelName('Git URL', githubURL); |
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.
I think variable should be renamed as well.
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.
Updated.
@@ -52,7 +52,7 @@ export class IntegrationTestsTabPage { | |||
this.verifySaveChangesIsDisabled(); | |||
|
|||
if (githubURL) { | |||
UIhelper.inputValueInTextBoxByLabelName('GitHub URL', githubURL); | |||
UIhelper.inputValueInTextBoxByLabelName('Git URL', githubURL); |
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.
Same here.
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.
Updated.
LGTM |
/retest |
/test |
/retest |
Replace GitHub mentions with Git - part2