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

Add support for non-whitespace opt-in "style fixes". #694

Merged
merged 1 commit into from Jun 8, 2018

Conversation

Projects
None yet
3 participants
@munificent
Member

munificent commented Jun 8, 2018

Implement the first fix, converting ":" to "=" as the named default
value separator.

Add support for non-whitespace opt-in "style fixes".
Implement the first fix, converting ":" to "=" as the named default
value separator.

@munificent munificent requested a review from natebosch Jun 8, 2018

@googlebot googlebot added the cla: yes label Jun 8, 2018

@natebosch

We chatted offline about potentially hiding the individual fixes from the --help since we don't want to encourage their use other than during a migration.

@munificent munificent merged commit 9eb5c5d into master Jun 8, 2018

2 checks passed

cla/google All necessary CLAs are signed
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@munificent munificent deleted the style-fixes branch Jun 8, 2018

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