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

Typo Fix: in CONTRIBUTING.md #1245

Merged
merged 4 commits into from
Sep 15, 2023
Merged

Typo Fix: in CONTRIBUTING.md #1245

merged 4 commits into from
Sep 15, 2023

Conversation

sadath-12
Copy link
Contributor

@sadath-12 sadath-12 commented Sep 11, 2023

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

@sadath-12 sadath-12 requested review from a team and pkosiec September 11, 2023 06:05
@huseyinbabal huseyinbabal added documentation Documetation is missing or incorrect community labels Sep 11, 2023
Copy link
Collaborator

@mszostok mszostok left a 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 👍

@kubeshop kubeshop deleted a comment from mszostok Sep 15, 2023
Copy link
Collaborator

@pkosiec pkosiec left a 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.

@@ -37,7 +37,7 @@ func NewMigrate() *cobra.Command {
Supported Botkube bot platforms for migration:
- Socket Slack
- Discord
- Mattermost
- Mattermost
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
- Mattermost
- Mattermost

@@ -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
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
- Mattermost
- Mattermost

@pkosiec
Copy link
Collaborator

pkosiec commented Sep 15, 2023

@sadath-12 please ping us once this PR is ready to review again 👍

@sadath-12
Copy link
Contributor Author

@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

Copy link
Collaborator

@pkosiec pkosiec left a comment

Choose a reason for hiding this comment

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

LGTM

sadath-12 and others added 4 commits September 15, 2023 14:29
Signed-off-by: sadath-12 <[email protected]>
Signed-off-by: syedsadath-17 <[email protected]>
Signed-off-by: syedsadath-17 <[email protected]>
@pkosiec pkosiec enabled auto-merge (squash) September 15, 2023 12:29
@pkosiec pkosiec disabled auto-merge September 15, 2023 12:46
@pkosiec pkosiec merged commit 1a9c02b into kubeshop:main Sep 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community documentation Documetation is missing or incorrect
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants