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

Correct formatting when no parameters. #60

Merged
merged 1 commit into from
Sep 16, 2020
Merged

Correct formatting when no parameters. #60

merged 1 commit into from
Sep 16, 2020

Conversation

thomasvl
Copy link
Member

@thomasvl thomasvl commented Aug 7, 2020

No description provided.

@thomasvl
Copy link
Member Author

Update the template maintain the exact whitespace from before if there are params.

Visually, there is an extra line (or two), but this should make it less likely to cause folks docs to appear out of date as the only things that will need updating were cases where there were no params, but the header was being accidentally output.

@alandonovan
Copy link

@c-parsons

@thomasvl
Copy link
Member Author

@c-parsons can you take a look again since I updated it to minimize the impact on things with params (left the original whitespacing they had).

@c-parsons c-parsons merged commit 247c209 into bazelbuild:master Sep 16, 2020
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.

3 participants