-
Notifications
You must be signed in to change notification settings - Fork 200
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
chore: format integration tests #3399
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lets merge this now. There are some remaining parts that may be a bit questionable but overall I think there is a lot more that are good. For the remaining bits we can open issues for them and tackle them at a later date.
I'm really not a fan of how the formatter is collapsing code sections separated with a blank line together if there's a comment between them. This severely hampers readability for me. |
@TomAFrench, ok, I'll fix it |
* master: chore: update `async-lsp` to v0.1.0 (#3489) fix: adding proving key initialization (#3322) chore(fmt): don't lose line breaks between comments (#3505) feat: Add LSP command to profile opcodes in vscode (#3496) chore: format integration tests (#3399) chore: Refactor parser tests (with concat!()) (#3500) chore: format function signature (#3495) chore: update `self_cell` to v1.0.2 (#3487) chore: Fix broken link (#3497) chore: update array formatting style (#3486) chore(fmt): add fn_call_width (#3480) chore: Fix undesirable line break before '{' in function signature with comments (#3473) fix: apply predicate to over/underflow checks (#3494)
Description
Additional Context
Documentation*
Check one:
PR Checklist
cargo fmt
on default settings.