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

Let Prettier infer the parser when it can #199

Merged
merged 1 commit into from
Jul 18, 2022
Merged

Let Prettier infer the parser when it can #199

merged 1 commit into from
Jul 18, 2022

Conversation

@lassik
Copy link
Owner Author

lassik commented Jul 1, 2022

@liuyinz Here's an alternative implementation of the approach you used in PR #195. It adds a bit more robustness (though we are still using a regexp). Can you check whether this works for you?

@lassik lassik force-pushed the lassik-prettier branch from ddf150b to b1a4972 Compare July 1, 2022 08:26
Alternative impelementation of PR #195

See also:

- Issue #175
- PR #190, #182
@lassik lassik force-pushed the lassik-prettier branch from b1a4972 to c446dae Compare July 1, 2022 08:51
@lassik
Copy link
Owner Author

lassik commented Jul 1, 2022

Sorry, I botched the initial PR. Now tested. Also added a debug message showing the full --file-info output.

@liuyinz
Copy link

liuyinz commented Jul 18, 2022

Tests good. Merge? @lassik

@lassik lassik merged commit 9e61704 into master Jul 18, 2022
@lassik lassik deleted the lassik-prettier branch July 18, 2022 10:14
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.

2 participants