Skip to content

Conversation

…12127)

* fix(fmt): account for ternary operators when estimating size

* fix(fmt): handle comments between inherited base contracts

* test: layout + base inheritance
* fix(fmt): account for ternary operators when estimating size

* fix(fmt): filter libs when recursing

* style: clippy

* test: wipe contracts before formatting

* test: explicitly test ignore
* fix(fmt): use nbsp if LHS-most element fits

* fix(fmt): use nbsp if LHS-most element fits

* reset cache inside helper
…ndry-rs#12147)

* fix(fmt): prioritize parenthesis breaks in emit and revert stmts

* fix(fmt): respect bracket spacing when printing commasetp without
delimiters
@grandizzy grandizzy changed the title [WIP] fix: Backport anvil, fmt and doc fixes to v1.4.2 fix: Backport anvil, fmt and doc fixes to v1.4.2 Oct 17, 2025
@grandizzy grandizzy marked this pull request as ready for review October 17, 2025 14:59
Copy link
Contributor

@0xrusowsky 0xrusowsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

despite this includes the alloy patch, maybe we can i clude the full alloy bump commit too?

@grandizzy
Copy link
Collaborator Author

despite this includes the alloy patch, maybe we can i clude the full alloy bump commit too?

would like to keep changes at min, which one you refer to?

@0xrusowsky
Copy link
Contributor

would like to keep changes at min, which one you refer to?

2c84e1c

@grandizzy
Copy link
Collaborator Author

would like to keep changes at min, which one you refer to?

2c84e1c

will follow-up in v1.4.3

@grandizzy grandizzy merged commit 828441d into foundry-rs:release-1.4.0 Oct 18, 2025
14 checks passed
@github-project-automation github-project-automation bot moved this to Done in Foundry Oct 18, 2025
@grandizzy grandizzy deleted the backport-fixes-v1.4.2 branch October 18, 2025 04:10
@grandizzy grandizzy self-assigned this Oct 20, 2025
@jenpaff jenpaff moved this from Done to Completed in Foundry Oct 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Completed

Development

Successfully merging this pull request may close these issues.

2 participants