-
Notifications
You must be signed in to change notification settings - Fork 151
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
chore: reduce more call to get platform #1255
chore: reduce more call to get platform #1255
Conversation
- we have get the Release can derive info from it. Signed-off-by: Wen Zhou <[email protected]>
removing calls and renaming the variable are two unrelated to each other changes, right? |
removing calls is the init idea |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ykaliuta The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
8d3a6c2
into
opendatahub-io:incubation
Basically we can remove GetPlatform API completely |
- we have get the Release can derive info from it. Signed-off-by: Wen Zhou <[email protected]> (cherry picked from commit 8d3a6c2)
- we have get the Release can derive info from it. Signed-off-by: Wen Zhou <[email protected]> (cherry picked from commit 8d3a6c2)
- we have get the Release can derive info from it. Signed-off-by: Wen Zhou <[email protected]> (cherry picked from commit 8d3a6c2)
- we have get the Release can derive info from it. Signed-off-by: Wen Zhou <[email protected]> (cherry picked from commit 8d3a6c2)
- we have get the Release can derive info from it. Signed-off-by: Wen Zhou <[email protected]> (cherry picked from commit 8d3a6c2)
Description
How Has This Been Tested?
Screenshot or short clip
Merge criteria