-
Notifications
You must be signed in to change notification settings - Fork 9
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
feat(format): respect max line width #242
Merged
Merged
Changes from 1 commit
Commits
Show all changes
47 commits
Select commit
Hold shift + click to select a range
e27f90c
[WIP] match current tests while using a buffer
a-frantz f74d717
[WIP]
a-frantz 164261a
Update builder.rs
a-frantz 0f691d7
feat: config max line length
a-frantz c364a45
Update max_line_length.rs
a-frantz 2852d28
revise: indentation config
a-frantz 165603a
Update post.rs
a-frantz f1f4dec
Merge branch 'main' into feat/line-width
a-frantz f2d6448
Merge branch 'main' into feat/line-width
a-frantz ccc1104
WIP
a-frantz e1898e5
Merge branch 'main' into feat/line-width
a-frantz af36d91
WIP
a-frantz ce3a454
revise: use constants for min and max max line len
a-frantz 3278597
revise: use a constant for max space indent
a-frantz 95d649c
revise: hardcode tabs and spaces in fewer places
a-frantz 163237e
chore: code clean up
a-frantz 283ae6a
WIP
a-frantz df6f7a2
WIP
a-frantz a238b95
WIP
a-frantz a574f09
chore: consistency
a-frantz bc9f288
Merge branch 'main' into feat/line-width
a-frantz caa29c8
chore: code cleanup
a-frantz 7509e66
revise: new PostToken (TempIndent)
a-frantz 3dc09c6
fix: handle edge case where stream is empty
a-frantz 7c0e191
WIP
a-frantz 259ea59
perf: use Rc for temp_indent
a-frantz a34c9e0
WIP
a-frantz 7199199
chore: fmt
a-frantz 2e77ba3
WIP
a-frantz 8f51518
WIP
a-frantz 6126627
chore: code clean up
a-frantz 28643d2
fix: special handling for no line breaks
a-frantz f195a78
fix: reset indent level
a-frantz 6bf613f
fix: simplify `IfExpr` formatting
a-frantz 9fb4691
feat: can_be_line_broken is exhaustive
a-frantz 51473d5
Merge branch 'main' into feat/line-width
a-frantz 6049417
chore: code cleanup
a-frantz d2c683e
Merge branch 'feat/line-width' of https://github.com/stjude-rust-labs…
a-frantz d16da2a
Update CHANGELOG.md
a-frantz b4d1020
fix: rework Config Builder
a-frantz f55d392
Update builder.rs
a-frantz 52a9984
chore: review feedback
a-frantz 5a42020
Update wdl-format/src/token.rs
a-frantz c0e446d
revise: more specific name for line spacing policy
a-frantz b3a2631
chore: review feedback
a-frantz c62b9c1
chore: review feedback
a-frantz 1c2b6c7
chore: review feedback
a-frantz File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
revise: new PostToken (TempIndent)
- Loading branch information
commit 7509e66619dc30e659f572e2fcb8cfa5c795a042
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
It's not clear to me from these comments why this is needed. Maybe there is additional documentation below where the command block is formatted, but it may be useful to expand here.