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

feat(forge): add max supported EVM version in compiler -vv #9129

Merged
merged 6 commits into from
Oct 17, 2024

Conversation

grandizzy
Copy link
Collaborator

@grandizzy grandizzy commented Oct 16, 2024

Motivation

closes #9125

  • show max supported EVM version when running forge compiler resolve -v
  • start from default and normalize on solc version
  • for vyper will always show default evm version
  • tests redacted as evm default version is a moving target

Solution

@grandizzy grandizzy marked this pull request as ready for review October 16, 2024 15:31
Copy link
Member

@zerosnacks zerosnacks left a comment

Choose a reason for hiding this comment

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

Pending minor change to let JSON output adhere to verbosity, other than that LGTM 👍

@grandizzy
Copy link
Collaborator Author

Pending minor change to let JSON output adhere to verbosity, other than that LGTM 👍

please recheck 588bc53 as there are more things changes

@grandizzy grandizzy requested a review from zerosnacks October 16, 2024 19:17
@grandizzy grandizzy requested a review from DaniPopes October 16, 2024 19:28
@grandizzy
Copy link
Collaborator Author

re-requesting review as there were some more changes re json output

@zerosnacks
Copy link
Member

zerosnacks commented Oct 16, 2024

Pushed a documentation fix + a minor change to make the JSON not include paths when ran with verbosity mode 0 cc @grandizzy

{
   "Solidity":[
      {
         "version":"0.8.4"
      }
   ]
}

in line with

Solidity:
- 0.8.4

When no verbosity flag is passed. Note the clearing of the paths happens after filtering as otherwise the version won't be reported correctly.

@zerosnacks zerosnacks changed the title feat(forge): add max supported EVM version in compiler -v feat(forge): add max supported EVM version in compiler -vv Oct 16, 2024
@grandizzy grandizzy merged commit d5f6e34 into foundry-rs:master Oct 17, 2024
21 checks passed
@grandizzy grandizzy deleted the issue-9125 branch October 17, 2024 03:35
rplusq pushed a commit to rplusq/foundry that referenced this pull request Nov 29, 2024
…s#9129)

* feat(forge): add max supported EVM version in compiler -v

* shorter message, displayed on -vv verbosity

* match on verbosity

* Respect verbosity in json, nicer json output

* Redact default EVM version in tests

* make --json output not output paths in verbosity mode 0, equivalent of non-verbose text mode

---------

Co-authored-by: zerosnacks <[email protected]>
@grandizzy grandizzy added T-feature Type: feature C-forge Command: forge labels Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-forge Command: forge T-feature Type: feature
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

feat(forge compiler): add EVM version alongside version output
3 participants