-
Notifications
You must be signed in to change notification settings - Fork 291
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
Typo Fix: in CONTRIBUTING.md #1245
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi! thanks for spotting that. Please just remove the empty lines and besides that it LGTM 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry @mszostok I accidentally removed your comment (wanted to remove my draft review which I didn't publish before) 🤦 It was about the space which should be removed and then we need to run make gen-docs-cli
.
cmd/cli/cmd/migrate.go
Outdated
@@ -37,7 +37,7 @@ func NewMigrate() *cobra.Command { | |||
Supported Botkube bot platforms for migration: | |||
- Socket Slack | |||
- Discord | |||
- Mattermost | |||
- Mattermost |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Mattermost | |
- Mattermost |
cmd/cli/docs/botkube_migrate.md
Outdated
@@ -14,7 +14,7 @@ This command will create a new Botkube Cloud instance based on your existing Bot | |||
Supported Botkube bot platforms for migration: | |||
- Socket Slack | |||
- Discord | |||
- Mattermost | |||
- Mattermost |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Mattermost | |
- Mattermost |
@sadath-12 please ping us once this PR is ready to review again 👍 |
@pkosiec Done , I have re-changed it to build the botkube named binary through command itself since while i was reading your docs in other places also there was mentioned to run botkube binary |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: sadath-12 <[email protected]>
Signed-off-by: sadath-12 <[email protected]>
Signed-off-by: syedsadath-17 <[email protected]>
Signed-off-by: syedsadath-17 <[email protected]>
Fixes issue #1244 In order to run botkube binary we have to modify command to be
go build -o botkube ./cmd/botkube-agent
in the CONTRIBUTING.md