Skip to content

[Improvement] change forge build sizes report to Bytes #7567

@radeksvarz

Description

@radeksvarz

Component

Forge

Have you ensured that all of these are up to date?

  • Foundry
  • Foundryup

What version of Foundry are you on?

forge 0.2.0 (dbc48ea 2024-04-03T00:17:03.697365187Z)

What command(s) is the bug in?

forge build

Operating System

Linux

Describe the bug

forge build --sizes lists values for size and margin in kB. It is unclear whether 1000x or 1024x

It would be more clear if listed in Bytes right aligned as the sizes are up to 5 digits anyway.

image

To become :

Compiler run successful!
| Contract                       | Size (B)  | Margin (B)  |
|--------------------------------|-----------|-------------|
| Address                        |        86 |      24 490 |
| SomeHugeContract               |    17 906 |       6 670 |

Metadata

Metadata

Assignees

No one assigned

    Labels

    T-featureType: featurefirst issueA good way to start contributing

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions