Skip to content

Remove unused Commands#146

Merged
zelima merged 1 commit into
masterfrom
remove-unused-commands
Jun 21, 2021
Merged

Remove unused Commands#146
zelima merged 1 commit into
masterfrom
remove-unused-commands

Conversation

@zelima

@zelima zelima commented Jun 17, 2021

Copy link
Copy Markdown
Collaborator

Removing Commands that are not used for now and are making unnecessary noise. I'm commenting them for now, but not deleting as they might be useful In future. Will bring them back as needed

@pdelboca pdelboca left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to merge @zelima!

As a personal preference I always try o delete instead of comment. Commented code blocks add noise and, if they are needed in the future, we can always come back to the PR and add them again.

@zelima

zelima commented Jun 21, 2021

Copy link
Copy Markdown
Collaborator Author

@pdelboca Agree I also prefer, but want to keep for now

1 similar comment
@zelima

zelima commented Jun 21, 2021

Copy link
Copy Markdown
Collaborator Author

@pdelboca Agree I also prefer, but want to keep for now

@zelima
zelima merged commit 588b363 into master Jun 21, 2021
@zelima
zelima deleted the remove-unused-commands branch June 21, 2021 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants