Skip to content

fix(docs): Add synopsis for bind apiservice#392

Merged
xrstf merged 1 commit into
kbind-dev:mainfrom
olamilekan000:add-synopsis-forapiservice
Nov 24, 2025
Merged

fix(docs): Add synopsis for bind apiservice#392
xrstf merged 1 commit into
kbind-dev:mainfrom
olamilekan000:add-synopsis-forapiservice

Conversation

@olamilekan000

@olamilekan000 olamilekan000 commented Nov 19, 2025

Copy link
Copy Markdown
Contributor

Summary

Change adds the description for the "bind apiservice" command, which was missing a synopsis and a detailed explanation.

What Type of PR Is This?

/kind bug

Related Issue(s)

Fixes 391

Release Notes

Added synopsis for bind apiservice command.

@olamilekan000 olamilekan000 requested a review from a team as a code owner November 19, 2025 18:28
@olamilekan000 olamilekan000 force-pushed the add-synopsis-forapiservice branch 2 times, most recently from 3229902 to 0f50bc7 Compare November 19, 2025 18:56
Comment thread cli/pkg/help/help.go Outdated
@olamilekan000 olamilekan000 force-pushed the add-synopsis-forapiservice branch from 0f50bc7 to db75eb0 Compare November 21, 2025 10:54
@olamilekan000 olamilekan000 force-pushed the add-synopsis-forapiservice branch from db75eb0 to 94df158 Compare November 21, 2025 11:02
coderabbitai[bot]
coderabbitai Bot previously approved these changes Nov 21, 2025
@xrstf

xrstf commented Nov 21, 2025

Copy link
Copy Markdown
Contributor

I think I found a simpler solution via #396. It's based on yours, but mostly supersedes it (besides the added help text your PR provides). :)

@olamilekan000

Copy link
Copy Markdown
Contributor Author

I think I found a simpler solution via #396. It's based on yours, but mostly supersedes it (besides the added help text your PR provides). :)

oh, that's cool!

@xrstf

xrstf commented Nov 21, 2025

Copy link
Copy Markdown
Contributor

Oh you could have rebased this, so your new help text makes it into kube-bind.

@olamilekan000

olamilekan000 commented Nov 21, 2025

Copy link
Copy Markdown
Contributor Author

Oh you could have rebased this, so your new help text makes it into kube-bind.

Yeah... I actually forgot about that part for a moment. Lol

@olamilekan000 olamilekan000 force-pushed the add-synopsis-forapiservice branch from 069a373 to 05f7324 Compare November 21, 2025 16:48
@olamilekan000 olamilekan000 changed the title fix(docs): Add synopsis for bind apiservice and preserve list formatting fix(docs): Add synopsis for bind apiservice Nov 21, 2025
@olamilekan000

Copy link
Copy Markdown
Contributor Author

I think I found a simpler solution via #396. It's based on yours, but mostly supersedes it (besides the added help text your PR provides). :)

@xrsf done!

Comment thread cli/pkg/kubectl/bind-apiservice/cmd/cmd.go Outdated
Signed-off-by: olalekan odukoya <odukoyaonline@gmail.com>
@olamilekan000 olamilekan000 force-pushed the add-synopsis-forapiservice branch from 05f7324 to 510bd35 Compare November 24, 2025 10:30
@olamilekan000 olamilekan000 requested a review from xrstf November 24, 2025 10:30

@xrstf xrstf 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.

jo

@xrstf xrstf merged commit 90f506f into kbind-dev:main Nov 24, 2025
7 checks passed
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.

bug(docs): Add synopsis for bind apiservice and fix list formatting

3 participants