Skip to content

chore: remove most editor mode config comments#1637

Merged
scop merged 1 commit into
mainfrom
chore/remove-most-mode-comments
Jun 1, 2026
Merged

chore: remove most editor mode config comments#1637
scop merged 1 commit into
mainfrom
chore/remove-most-mode-comments

Conversation

@scop
Copy link
Copy Markdown
Owner

@scop scop commented May 30, 2026

In particular, remove them from files with names containing a typical extension for a language. These should be unnecessary at best, but an unknown mode or alias can also prevent a file from being opened with the correct mode.

This is what happens currently with Zed 1.4.4, ref zed-industries/zed#58124

Giant but trivial/comment-only PR.

Copy link
Copy Markdown
Collaborator

@yedayak yedayak left a comment

Choose a reason for hiding this comment

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

I don't really have the context of what uses these comments, but looks good to me

Copy link
Copy Markdown
Collaborator

@akinomyoga akinomyoga left a comment

Choose a reason for hiding this comment

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

I've thought GNU Emacs didn't recognize .bash, but I realized that the current version of Bash recognizes it. Perhaps it has recently added support, or I misunderstood.

In particular, remove them from files with names containing a typical
extension for a language. These should be unnecessary at best, but an
unknown mode or alias can also prevent a file from being opened with
the correct mode.

This is what happens currently with Zed 1.4.4, ref
zed-industries/zed#58124
@scop scop force-pushed the chore/remove-most-mode-comments branch from 871bc5e to ee9eab5 Compare June 1, 2026 18:26
@scop scop merged commit bdeca44 into main Jun 1, 2026
11 checks passed
@scop scop deleted the chore/remove-most-mode-comments branch June 1, 2026 18:26
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