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

feat(cmd): add aliases for database user management #1031

Merged

Conversation

curzolapierre
Copy link
Member

@curzolapierre curzolapierre commented Jan 24, 2024

Fixes #1030

  • Add a changelog entry in the section "To Be Released" of CHANGELOG.md

@curzolapierre curzolapierre self-assigned this Jan 24, 2024
@curzolapierre curzolapierre force-pushed the feat/1030/db_users_management_command_consistency branch from 982db6d to 0e322f7 Compare January 24, 2024 10:10
@curzolapierre curzolapierre marked this pull request as ready for review January 24, 2024 10:11
Copy link

cmd/databases.go Outdated
@@ -142,6 +142,7 @@ var (

databaseListUsers = cli.Command{
Name: "database-list-users",
Aliases: []string{"database-users-list"},
Copy link
Contributor

Choose a reason for hiding this comment

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

nitpick (non-blocking): as "database-users-" if our wanted prefix, shouldn't it be the name of the command? and the old name the alias

Copy link
Member Author

Choose a reason for hiding this comment

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

Yep you're right

@curzolapierre curzolapierre force-pushed the feat/1030/db_users_management_command_consistency branch from 0e322f7 to 17840fc Compare January 24, 2024 13:09
Copy link
Contributor

@yohann-bacha yohann-bacha left a comment

Choose a reason for hiding this comment

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

If you accepted my nitpick though, you must update the examples as well

@curzolapierre curzolapierre force-pushed the feat/1030/db_users_management_command_consistency branch from 17840fc to 7c1903c Compare January 24, 2024 13:34
Copy link
Contributor

@yohann-bacha yohann-bacha left a comment

Choose a reason for hiding this comment

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

LGTM

@curzolapierre curzolapierre force-pushed the feat/1030/db_users_management_command_consistency branch from 7c1903c to 2eecaf8 Compare January 24, 2024 13:45
@curzolapierre curzolapierre merged commit dc45956 into master Jan 24, 2024
5 checks passed
@curzolapierre curzolapierre deleted the feat/1030/db_users_management_command_consistency branch January 24, 2024 13:47
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.

[Database Users] Create aliases to be more consitent with naming
2 participants