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 tool to "enforce" semantic line breaks #1485

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tshepang
Copy link
Member

@tshepang tshepang commented Oct 18, 2022

See #1132

Run it so:

cd ci/semantic-line-breaks
cargo run --- ../../src [--overwrite]

Diff is large, so maybe should be automated.

@JohnTitor
Copy link
Member

I don't have an objection to enforcing sembr but the implementation does look fragile, e.g. there's no tokenizer. Have you considered using an existing tool like https://github.com/bobheadxi/readable?

@tshepang
Copy link
Member Author

I have not... thanks for the suggestion, will have a look.

@JohnTitor JohnTitor added the waiting-on-author This PR is waiting for additional action by the OP label Oct 22, 2022
@m-spitfire
Copy link

m-spitfire commented Jan 23, 2023

I have tried using readable to have a GitHub action that runs check, and format all the existing code, but the tool is immature, it gave errors for some files saying Cannot take length of multi-line node '[object Object]' (bobheadxi/readable#18) and for others the aforementioned issue is there, it does the wrong thing when it encounters e.g.(bobheadxi/readable#17). I am not sure what should be done here, make readable better or create another tool?

@tshepang
Copy link
Member Author

This tool of mine handles both issues without choking. I intend to get back to making it more fancy... I keep postponing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
waiting-on-author This PR is waiting for additional action by the OP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants