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

Fix 'wrap-comments' not working with the janestreet profile #2645

Merged
merged 4 commits into from
Jan 8, 2025

Conversation

Julow
Copy link
Collaborator

@Julow Julow commented Jan 7, 2025

Fix #2642
Fix a bug added in #2371

Julow added 2 commits January 7, 2025 16:34
Asterisk-prefixed comments were recognized as documentation with the
janestreet profile and the wrap-comments option didn't have an effect.
@Julow
Copy link
Collaborator Author

Julow commented Jan 7, 2025

Also fix #2643

@Julow
Copy link
Collaborator Author

Julow commented Jan 8, 2025

I can only spot differences on asterisk-prefixed comments in test-branch. Let's merge!

@Julow Julow merged commit 4c94d48 into ocaml-ppx:main Jan 8, 2025
5 of 7 checks passed
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.

Question: What does wrap-comments do?
1 participant