Tejaskriya

Results 14 comments of Tejaskriya

@sodonnel @siddhantsangwan Thank you for the reviews! I have addressed the comments left. The check is now performed if the --force flag is not set. Could you please take a...

> @Tejaskriya Is this ready for review? @siddhantsangwan it is ready for review now. I have added few more checks in the same test cases with force flag set to...

@ChenSammi I have used translate app to add a Chinese version of the document. Could you please help me with verifying if the translation is done correctly?

@sadanand48 thanks for the review, I have addressed all the comments and made the changes needed.

@ChenSammi thanks for the review! I have updated the PR with the suggested changes. Could you please take a look again?

Thanks for the review @ChenSammi ! I have addressed all the comments and made the changes. Could you please approve the workflows?

@siddhantsangwan Thank you for finding the hack to test it! It works, I have added it in the unit tests. @siddhantsangwan @symious Could you please review this updated PR? I...

@siddhantsangwan Thank you for the review! I have made the changes suggested by you and addressed the remaining comments with appropriate code changes. Could you please take another look at...

@siddhantsangwan @sumitagrawl following our discussion today, I have reverted the latest commit. Now the container is excluded for all DNs as ContainerNotFoundException will come up for every DN that the...

> Sure, I have plans to finish it in one or two PRs (@siddhantsangwan has told first time that too many lines per one PR). I'd like to finish in...