-
Notifications
You must be signed in to change notification settings - Fork 138
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
refactor: Update application-api references #1174
refactor: Update application-api references #1174
Conversation
Skipping CI for Draft Pull Request. |
922487d
to
7af787c
Compare
The PR checks are now failing because konflux-ci/application-api#83 needs to be merged. |
/retest |
@johnbieren yes, I think redhat-appstudio/application-service#476 needs to be merged first. Added WIP prefix. |
7ca0b5a
to
77c061a
Compare
Quality Gate passedIssues Measures |
1e09504
to
a15d7f0
Compare
a15d7f0
to
d908955
Compare
d908955
to
5ca500e
Compare
@dheerajodha I had to rebase my PR. I've just pushed the updated version. Could you please review once more? Thanks! |
@thepetk Sure, I'm seeing some conflicts with the |
Done! I have lost the count on the times I rebased :D Thanks for the help! |
/test redhat-appstudio-e2e |
Signed-off-by: thepetk <[email protected]>
Signed-off-by: thepetk <[email protected]>
Signed-off-by: thepetk <[email protected]>
Signed-off-by: thepetk <[email protected]>
Signed-off-by: thepetk <[email protected]>
Signed-off-by: thepetk <[email protected]>
78f488a
to
f63ee2f
Compare
@dheerajodha I've rebased once more, I feel the faster we merge it the better :D |
/lgtm |
Ah sorry about that, idky it gave you this rebase hell, this doesn't usually happen :D but merged it finally 🎉 |
Description
The PR updates all references to
application-api
using the new organizationkonflux-ci
instead ofredhat-appstudio
.It also removes any mentions to
DeploymentTarget
as it was removed hereIssue ticket number and link
https://issues.redhat.com/browse/DEVHAS-656
Type of change
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist: