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

Readme: add note about configuring autoformat when config is present #317

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jonpacker
Copy link

Summary

Adds a simple note to the autoformat_config_present section documenting that the setting does nothing without switching off autoformat_require_pragma first. I found this confusing when setting up this plugin myself, and the reactions in #191 seem to indicate I wasn't alone 😅

@TimDaub
Copy link

TimDaub commented Mar 10, 2022

Actually, I think this comment #191 (comment) is instructive and should also be added (with slight modification) to the readme.md.

@jtheoof
Copy link

jtheoof commented May 19, 2022

I was configuring vim-prettier today and run into the issue until I saw the link posted by @TimDaub and found this MR.

Really feel like this MR should be merged but I agree it would be even better with the comment posted.

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