Skip to content

Replace spaces with tabs in mixed whitespace string - #6456

Merged
samcoe merged 1 commit into
cli:trunkfrom
rneatherway:tabs-to-spaces
Oct 18, 2022
Merged

samcoe merged 1 commit into
cli:trunkfrom
rneatherway:tabs-to-spaces

Conversation

@rneatherway

Copy link
Copy Markdown
Contributor

This makes the indentation of the query consistent no matter your local tab size rendering.

I was reading this query and was baffled until I realised the indentation was just rendered incorrectly. This is how it looks for me before this PR:

image

This makes the indentation of the query consistent no matter your
local tab size rendering.
@rneatherway
rneatherway requested a review from a team as a code owner October 18, 2022 10:59
@rneatherway
rneatherway requested review from samcoe and removed request for a team October 18, 2022 10:59
@cliAutomation cliAutomation added the external pull request originating outside of the CLI core team label Oct 18, 2022
@cliAutomation

Copy link
Copy Markdown
Contributor

Hi! Thanks for the pull request. Please ensure that this change is linked to an issue by mentioning an issue number in the description of the pull request. If this pull request would close the issue, please put the word 'Fixes' before the issue number somewhere in the pull request body. If this is a tiny change like fixing a typo, feel free to ignore this message.

@samcoe samcoe left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks!

@samcoe
samcoe enabled auto-merge (squash) October 18, 2022 11:07
@samcoe
samcoe merged commit 0197b51 into cli:trunk Oct 18, 2022
@rneatherway
rneatherway deleted the tabs-to-spaces branch October 18, 2022 11:08
@gianvi074

This comment was marked as spam.

@mfghelth

This comment was marked as spam.

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

Labels

external pull request originating outside of the CLI core team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants