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

zero config: add disable svcname generation capability #2410

Merged
merged 3 commits into from
Jan 3, 2023

Conversation

pmcollins
Copy link
Contributor

Service name generation can now be disabled by setting generate_service_name=false in
the config file. In a later release, service name generation will be disabled by
default.

This change also adds a generate_svc_name config param.
This change also adds a generate_svc_name config param.
setting service name can be disabled by setting generate_service_name=false in
the config file. in a later release service name generation will be disabled by
default.
return v != NULL && !streq("false", v) && !streq("FALSE", v) && !streq("0", v);
}

bool str_eq_false(char *v) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm going to consolidate these two functions into str_to_bool(char *v, bool default) in a separate PR.

Copy link
Contributor

@atoulme atoulme left a comment

Choose a reason for hiding this comment

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

LGTM, can merge next year :)

@pmcollins pmcollins marked this pull request as ready for review January 3, 2023 18:14
@pmcollins pmcollins requested review from a team as code owners January 3, 2023 18:14
@pmcollins pmcollins merged commit 4db006c into main Jan 3, 2023
@delete-merged-branch delete-merged-branch bot deleted the zc-java-disable-svcname branch January 3, 2023 18:15
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.

None yet

2 participants