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

[#103] Specify --parser in Prettier #107

Merged
merged 8 commits into from
Sep 11, 2022
Merged

[#103] Specify --parser in Prettier #107

merged 8 commits into from
Sep 11, 2022

Conversation

raxod502
Copy link
Member

Closes #103

@lassik
Copy link

lassik commented Jun 27, 2022

See also prettier/prettier#13052

@elken
Copy link
Contributor

elken commented Aug 16, 2022

WDYT about the linked PR @raxod502 ?

This should also be expanded to handle GraphQL and Markdown

@raxod502
Copy link
Member Author

The linked PR has an interesting idea. Since we already have to explicitly support each new major mode, I am not sure if it will actually improve the user experience to adopt that approach versus what I have here. Let me add Markdown and GraphQL as you suggested (subject to #81), then merge this.

@raxod502
Copy link
Member Author

Filed ruby-syntax-tree/syntax_tree#144 for an upstream issue I discovered while trying to get CI to pass.

@elken
Copy link
Contributor

elken commented Aug 22, 2022

Oh how annoying!

@raxod502
Copy link
Member Author

raxod502 commented Sep 3, 2022

The folks upstream most generously fixed that issue, but apparently the same problem occurs with https://github.com/ruby-syntax-tree/syntax_tree-rbs, which is also a dependency, so I'll submit a pull request for that.

@raxod502
Copy link
Member Author

raxod502 commented Sep 3, 2022

I filed ruby-syntax-tree/syntax_tree-rbs#34, but my Ruby inexperience is showing, unfortunately.

@raxod502
Copy link
Member Author

raxod502 commented Sep 3, 2022

Okay, I fixed up ruby-syntax-tree/syntax_tree-rbs#34 and confirmed that it fixes the CI here, so after that PR is merged, I'll go ahead and rebuild this and it should be good to go.

If it takes a long time to merge the upstream PR, then I can adjust the CI here to pull from my branch for now.

@raxod502
Copy link
Member Author

Sorry this took so long. But it's done now!

@raxod502 raxod502 merged commit 46d373f into main Sep 11, 2022
@raxod502 raxod502 deleted the rr-shard-prettier branch September 11, 2022 00:24
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.

prettier formatter error: find no parser
3 participants