Skip to content
This repository was archived by the owner on Dec 12, 2023. It is now read-only.

comment out the install and monitoring code#127

Merged
shrimalmadhur merged 4 commits into
developfrom
madhur/remove-install-cli-options
Oct 24, 2023
Merged

comment out the install and monitoring code#127
shrimalmadhur merged 4 commits into
developfrom
madhur/remove-install-cli-options

Conversation

@shrimalmadhur
Copy link
Copy Markdown
Collaborator

@shrimalmadhur shrimalmadhur commented Oct 24, 2023

Fixes | Closes | Resolves #

We are commenting out the install and monitoring option right now and will evaluate these as we get more feedback from operators

We are not deleting it since we might need to bring it back again.

Changes:

  • List the changes introduced by this PR more in detail here

Types of changes

Leave on the following list the types of changes introduced by this PR and remove
the ones that don't apply. Please also remove this line.

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update

Testing

Requires testing Yes/No

In case you checked yes, did you write tests? Yes/No

Comments about testing , should you have some (optional)

Further comments (optional)

If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

Copy link
Copy Markdown

@bowenli86 bowenli86 left a comment

Choose a reason for hiding this comment

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

nice!

@shrimalmadhur shrimalmadhur merged commit 999df8f into develop Oct 24, 2023
@shrimalmadhur shrimalmadhur deleted the madhur/remove-install-cli-options branch October 24, 2023 17:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants