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.

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

Add a vim modeline to set textwidth to 100. #44

Closed
wants to merge 1 commit into from
Closed

Add a vim modeline to set textwidth to 100. #44

wants to merge 1 commit into from

Conversation

sideshowbarker
Copy link
Contributor

No description provided.

@domenic
Copy link
Member

domenic commented Aug 29, 2015

Doesn't a comment inside a comment break stuff?

I'd normally say that we should just implement #38 instead, but it appears there's already a similar comment (ecmacs?), so it can't hurt...

@sideshowbarker
Copy link
Contributor Author

Doesn't a comment inside a comment break stuff?

Ah yes it does; I had not looked closely at the first line and had thought it had a closing comment delimiter at the end. Will amend my change.

I'd normally say that we should just implement #38 instead

Yeah but for users, that .editorconfig approach requires them to have some other plugin or something installed for their vim or emacs or sublime, doesn’t it?

but it appears there's already a similar comment (ecmacs?), so it can't hurt...

Yeah that was might thinking as well.

@domenic
Copy link
Member

domenic commented Aug 29, 2015

You keep leaving the periods at the end of your commit messages :P. LGTM, merge at will. And yeah, .editorconfig requires a plugin.

@sideshowbarker
Copy link
Contributor Author

You keep leaving the periods at the end of your commit messages

oofs. Old habits hard to break and all that. Will amend and merge.

@sideshowbarker sideshowbarker deleted the vim-modeline branch August 29, 2015 04:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants