Closed ansijain closed 2 years ago
Hi @ansijain. Thanks for your PR.
I'm waiting for a kubevirt member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test
on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test
label.
I understand the commands that are listed here.
/ok-to-test
@ansijain Do you think you can make the page a template that sources a data file? See index.html and adopters.yml
@ansijain Do you think you can make the page a template that sources a data file? See index.html and adopters.yml
Well yes. I have added a new commit regarding this. Please check
/retest
I fixed a couple problems so we could get moving along. That target repo updated their main branch and broke our link. The link checker checks production website for new urls. That creates a chicken/egg scenario with new website urls. I just set to ignore. We can remove the ignore in the future.
The new addition looks great otherwise! Thanks!
/lgtm /approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mazzystr
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Signed-off-by: ansijain ansi.jain@india.nec.com
What this PR does / why we need it: Adding Kubevirt Summit playlist to videos section.
Does this PR fix any issue? _(optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged)_:Special notes for your reviewer: