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: update to Soldeer v0.4.0 #9014

Merged
merged 10 commits into from
Oct 8, 2024
Merged

Conversation

mario-eth
Copy link
Contributor

Motivation

We finally have the version 0.4.0 ready.
In 0.4.0 we kinda did a rewrite to have it more rust idiomatic and solved also bugs that were waiting to be fixed.

mario-eth/soldeer#202
Full description of the release!

Important mention:
we did split the project into 3 crates:

  • cli - used only for soldeer alone
  • commands - used for commands alone
  • core - core crate

We did not publish the crates yet as we are waiting for feedback here beforehand.

Solution

Copy link
Member

@mattsse mattsse left a comment

Choose a reason for hiding this comment

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

this seems fine, using commands directly makes sense

Copy link
Contributor

@beeb beeb left a comment

Choose a reason for hiding this comment

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

Would love to see this merged before I leave on vacation end of next week!

@DaniPopes DaniPopes changed the title [DRAFT] Update to Soldeer v0.4.0 feat: update to Soldeer v0.4.0 Oct 7, 2024
@DaniPopes
Copy link
Member

DaniPopes commented Oct 7, 2024

Blocked on mario-eth/soldeer#202 + release

@DaniPopes DaniPopes added the T-blocked Type: blocked label Oct 7, 2024
Comment on lines 27 to 35
◆ Done reading config

◇ Installing forge-std~1.8.1
◆ Dependency added to config
◆ Dependency added to lockfile
◆ Dependency added to remappings
Copy link
Member

Choose a reason for hiding this comment

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

looks like we need to escape ◇ here

Copy link
Contributor

@beeb beeb Oct 7, 2024

Choose a reason for hiding this comment

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

Removing the control characters before the diamonds also works, but yeah probably easier to just wildcard it

@mattsse mattsse merged commit 0b9bdf3 into foundry-rs:master Oct 8, 2024
22 checks passed
@DaniPopes DaniPopes removed the T-blocked Type: blocked label Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants