Skip to content

Conversation

@chenrui333
Copy link
Contributor

📝 Description

fix completion generation

closes #607

✔️ How to Test

Signed-off-by: Rui Chen <rui@chenrui.dev>
@chenrui333 chenrui333 requested a review from a team as a code owner May 1, 2024 22:38
@chenrui333 chenrui333 requested review from jriddle-linode and lgarber-akamai and removed request for a team May 1, 2024 22:38
@zliang-akamai
Copy link
Member

Hi @chenrui333, thank you so much for the PR! Can you do a make format? Or I can help and push a commit to your branch.

Signed-off-by: Rui Chen <rui@chenrui.dev>
@chenrui333
Copy link
Contributor Author

should be good now.

Copy link
Contributor

@lgarber-akamai lgarber-akamai left a comment

Choose a reason for hiding this comment

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

Tested locally by removing my local config, unsetting LINODE_CLI_TOKEN, and running linode-cli completions bash. Thank you for the contribution!

@jriddle-linode jriddle-linode added the improvement for improvements in existing functionality in the changelog. label May 2, 2024
@chenrui333
Copy link
Contributor Author

would be good to make a followup patch release :)

@lgarber-akamai lgarber-akamai merged commit 544ba0d into linode:dev May 2, 2024
@ykim-akamai
Copy link
Contributor

would be good to make a followup patch release :)

Patch released with v5.49.1. Thanks for the contributions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement for improvements in existing functionality in the changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

completion command should not require PAT token

5 participants