Skip to content

EditorConfig: don't replace tabs with spaces in tsv files #15815

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

Merged
merged 1 commit into from
Jul 22, 2021
Merged

EditorConfig: don't replace tabs with spaces in tsv files #15815

merged 1 commit into from
Jul 22, 2021

Conversation

SethFalco
Copy link
Contributor

PR Summary

Adds a rule to the EditorConfig file for TSV files.

PR Context

Prevent tabs from being converted to spaces when updating or formatting the TSV file.

PR Checklist

Copy link
Collaborator

@iSazonov iSazonov left a comment

Choose a reason for hiding this comment

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

We have one such file CounterSamples.tsv

@iSazonov iSazonov added the CL-Tools Indicates that a PR should be marked as a tools change in the Change Log label Jul 22, 2021
@iSazonov iSazonov changed the title chore: don't replace tabs with spaces in tsv files EditorConfig: don't replace tabs with spaces in tsv files Jul 22, 2021
@iSazonov iSazonov merged commit 0002f7e into PowerShell:master Jul 22, 2021
@iSazonov
Copy link
Collaborator

@SethFalco Thanks for your contribution.

@iSazonov iSazonov added this to the 7.2.0-preview.8 milestone Jul 22, 2021
@ghost
Copy link

ghost commented Aug 23, 2021

🎉v7.2.0-preview.9 has been released which incorporates this pull request.:tada:

Handy links:

@ghost
Copy link

ghost commented Sep 28, 2021

🎉v7.2.0-preview.10 has been released which incorporates this pull request.:tada:

Handy links:

@SethFalco SethFalco deleted the patch-1 branch August 24, 2024 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CL-Tools Indicates that a PR should be marked as a tools change in the Change Log
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants