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.

Sign up for GitHub

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

Jump to bottom

fix: use standard-indent instead of tab-width #335

Merged
merged 1 commit into from
Sep 6, 2024
Merged

fix: use standard-indent instead of tab-width #335

merged 1 commit into from
Sep 6, 2024

Commits on Aug 31, 2024

  1. fix: use standard-indent instead tab-width for indentation 

    I see the following warnings.

```
Warning: copilot--infer-indentation-offset found no mode-specific indentation offset, using ’tab-width’ instead.  You can suppress this error message by customizing ’copilot-indent-warning-suppress’.
```

Reading the documentation comments for `tab-width` shows that it has no effect if spaces are used for indentation.

```
Documentation
Distance between tab stops (for display of tab characters), in columns.

This controls the width of a TAB character on display.
The value should be a positive integer.
Note that this variable doesn't necessarily affect the size of the
indentation step.  However, if the major mode's indentation facility
inserts one or more TAB characters, this variable will affect the
indentation step as well, even if indent-tabs-mode is non-nil.
```

If tabs are used, the `tab-width` is automatically adjusted, so there seems to be no need to worry about it.

Conversely, `tabSize` is supposed to be the real tab size, not the indentation size, so put the value as it is.
    @ncaq
    ncaq committed Aug 31, 2024
    Configuration menu
    Copy the full SHA
    72ac93c View commit details
    Browse the repository at this point in the history