Create convention for organization of CLI helper functions about airshipctl HOT 7 CLOSED

@sirajyasin is working on this issue. i am not able to assign it to him. can some one help me in assigning this to him?

from airshipctl.

@sreejithpunnapuzha I'm unable to add him as well. I've done some digging into why this keeps happening, and it seems as though Github disallows assigning issues to arbitrary users to prevent abuse. In order to assign a user, that user must either have write permissions to the repo (as of now, I believe this is limited to WC and cores), or they must have left a comment on the issue.

@sirajyasin if you would like me to assign you to this issue, please leave a comment here.

from airshipctl.

@ian-howell , thanks for looking into the GitHub issue.
Yes, please assign it to me.

from airshipctl.

Related Change #713755

Subject: Organize CLI helper functions
Link: https://review.opendev.org/713755
Status: MERGED
Owner: Sirajudeen ([email protected])

Approvals

Code-Review
+2 Alexander Hughes
+2 Ian Howell
Verified
+2 Zuul
Workflow
+1 Ian Howell

Last Updated: 2020-04-22 16:45:08 CDT

from airshipctl.

Removed the cmds.go within config/ and added a helper for config ( config_helper.go ). If this approach is agreed, then we can create helpers for respective packages when required and have the additional validations or any kind of helper functions done there

PS for review in Gerrit

from airshipctl.

The commit is merged. So this issue can be marked closed. Going further i will follow the comment convention to auto close.

from airshipctl.

A [Related Change](https://review.opendev.org/778359 was merged. This issue may be ready to close.

from airshipctl.