Skip to content

Update README.md: consistent indentation #967

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

Closed
wants to merge 1 commit into from
Closed

Update README.md: consistent indentation #967

wants to merge 1 commit into from

Conversation

caldwell-63
Copy link

Makes indentation consistent across the entire file

Makes indentation consistent across the entire file
@romainmenke
Copy link
Member

romainmenke commented May 13, 2023

Hi @caldwell-63
Thank you for your contribution!

Can you elaborate a bit on what you are trying to solve?

For context, these examples are actual output from the plugin and the plugin doesn't alter indents.

Why did you also change indents to spaces?

@romainmenke
Copy link
Member

Closing because this is an unwanted change.

Feel free open an issue to discuss any problems you might be trying to solve here :)

@caldwell-63
Copy link
Author

8-width tabs in readme look absolutely horrendous, therefore this pull request

@romainmenke
Copy link
Member

romainmenke commented May 15, 2023

Hi @caldwell-63

We always use tabs exactly because anyone can configure their preferred tab size.
If yours has a width of 8 then it is because you are using a default or have set it to have a width of 8.

For GitHub you can find more info here: https://docs.github.com/en/account-and-profile/setting-up-and-managing-your-personal-account-on-github/managing-personal-account-settings/managing-your-tab-size-rendering-preference#

We will never go back to spaces :)

@caldwell-63
Copy link
Author

Hey @romainmenke, thanks for the response

I do have the tab setting set to 2 indeed, but it simply does not work on readme files 😣

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

Successfully merging this pull request may close these issues.

2 participants