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
adds spec for RevisionRow component #3833
adds spec for RevisionRow component #3833
Conversation
e7fa628
to
71e91c5
Compare
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.
/lgtm
import { TableData } from '@console/internal/components/factory'; | ||
import { ResourceLink } from '@console/internal/components/utils'; | ||
import { K8sResourceConditionStatus } from '@console/internal/module/k8s'; | ||
import { revisionObj } from '@console/dev-console/src/components/topology/__tests__/topology-knative-test-data'; |
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.
It is wrong dependency. knative-plugin should have no dependency on dev-console packages. Not for this PR but create a task to fix this dependency issue.
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.
We do have this dependency right now. Knative Plugin depends on dev-console to render stuff in topology and a lot of other such instances. We probably need to come up with a runtime extensibility framework to fully eliminate the dependency.
/assign @rohitkrai03 @andrewballantyne |
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.
/approve
import { TableData } from '@console/internal/components/factory'; | ||
import { ResourceLink } from '@console/internal/components/utils'; | ||
import { K8sResourceConditionStatus } from '@console/internal/module/k8s'; | ||
import { revisionObj } from '@console/dev-console/src/components/topology/__tests__/topology-knative-test-data'; |
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.
We do have this dependency right now. Knative Plugin depends on dev-console to render stuff in topology and a lot of other such instances. We probably need to come up with a runtime extensibility framework to fully eliminate the dependency.
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: invincibleJai, rohitkrai03, vikram-raj 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 |
/retest Please review the full test history for this PR and help us cut down flakes. |
2 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/hold |
/kind cleanup |
/hold cancel |
/retest Please review the full test history for this PR and help us cut down flakes. |
adds spec for RevisionRow component
Tracks: https://issues.redhat.com/browse/ODC-2640