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.

Sign up for GitHub

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

Jump to bottom

WFLY-16960 Change single quote to double quote in docs cli sample com… #16027

Merged
merged 1 commit into from Sep 8, 2022
Merged

WFLY-16960 Change single quote to double quote in docs cli sample com… #16027

merged 1 commit into from Sep 8, 2022

Conversation

chengfang
Copy link
Contributor

@github-actions github-actions bot added the deps-ok Dependencies have been checked, and there are no significant changes label Sep 7, 2022
@bstansberry bstansberry merged commit e03e30e into wildfly:main Sep 8, 2022
@bstansberry
Copy link
Contributor

Thanks @chengfang

@shipsing
Copy link

shipsing commented Sep 8, 2022

@chengfang, don't we need to use double quotes for database and partition names as both the parameters represent a string? Asking this question for better understanding about the usage of quotes.

Thanks

@chengfang
Copy link
Contributor Author

@shipsing CLI attribute types (string, number, etc) are automatically handled without need for quoting. Double quote can be used to escape some special characters, or when it's no certain.

@chengfang chengfang deleted the cli.quote.WFLY-16960 branch September 8, 2022 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
deps-ok Dependencies have been checked, and there are no significant changes
Projects
None yet
Milestone
No milestone
3 participants
@chengfang @bstansberry @shipsing