Skip to content
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

cleanup: removes the docs from README to docs folder #124

Merged
merged 1 commit into from
Mar 29, 2022

Conversation

yati1998
Copy link
Contributor

This commit removes the installations methods of CSI-Addons
controller from the README file to deploy-controller.md inside
the docs folder. This will keep the README clean and have all
documentation at one place.

Signed-off-by: yati1998 [email protected]

@yati1998 yati1998 requested a review from nixpanic February 28, 2022 11:11
@mergify mergify bot requested review from Rakshith-R and Yuggupta27 February 28, 2022 11:11
@yati1998
Copy link
Contributor Author

This PR would require #123 to get merged first.

@nixpanic nixpanic added the documentation Improvements or additions to documentation label Feb 28, 2022
README.md Outdated Show resolved Hide resolved
docs/deploy-controller.md Outdated Show resolved Hide resolved
docs/deploy-controller.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
docs/deploy-controller.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
docs/deploy-controller.md Outdated Show resolved Hide resolved
docs/deploy-controller.md Outdated Show resolved Hide resolved
docs/deploy-controller.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@nixpanic nixpanic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just a few nits to correct. thanks!

nixpanic
nixpanic previously approved these changes Mar 15, 2022

* The "rbac.yaml" creates the required rbac.

* The setup-controller creates the csi-addons-controller-manager.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

setup-controller needs to be quoted as well?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and it should be setup-controller.yaml?

This commit removes the installations methods of CSI-Addons
controller from the README file to deploy-controller.md inside
the docs folder. This will keep the README clean and have all
documentation at one place.

Signed-off-by: yati1998 <[email protected]>
@yati1998 yati1998 requested review from Madhu-1 and nixpanic March 29, 2022 04:12
@mergify mergify bot dismissed nixpanic’s stale review March 29, 2022 04:12

Pull request has been modified.

@@ -164,7 +117,6 @@ deployment.apps/csi-addons-controller-manager created

The [Contribution Guidelines](CONTRIBUTING.md) contain details on the process
to contribute to this project.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this change required?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not required, just removed the extra space.

@mergify mergify bot merged commit 0109f96 into csi-addons:main Mar 29, 2022
Nikhil-Ladha pushed a commit to Nikhil-Ladha/kubernetes-csi-addons that referenced this pull request May 27, 2024
Syncing latest changes from upstream main for kubernetes-csi-addons
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants