Skip to content

Conversation

TilakMaddy
Copy link

Motivation

It would help application crates access the field for processing

Solution

Add a visibility modifier

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking changes

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.

I think we also want a deref impl for this

@jenpaff jenpaff moved this to Ready For Review in Foundry Apr 15, 2025
Copy link
Member

@DaniPopes DaniPopes left a comment

Choose a reason for hiding this comment

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

This is fine

@DaniPopes DaniPopes enabled auto-merge (squash) August 12, 2025 09:52
DaniPopes added a commit that referenced this pull request Aug 12, 2025
This PR aggregates changes from the following PRs:

- #10038: Make Soldeer Dependency field public by @TilakMaddy
- #11139: upgrade to Node.js 24 by @Daulox92
- #11274: CI: migrate workflows to checkout v5 by @rejected-l
- #11246: fix: add missing semicolon in typos error message in Makefile by @dizer-ti
DaniPopes added a commit that referenced this pull request Aug 12, 2025
This PR aggregates changes from the following PRs:

- #10038: Make Soldeer Dependency field public by @TilakMaddy
- #11139: upgrade to Node.js 24 by @Daulox92
- #11274: CI: migrate workflows to checkout v5 by @rejected-l
- #11246: fix: add missing semicolon in typos error message in Makefile by @dizer-ti
@DaniPopes DaniPopes closed this Aug 12, 2025
auto-merge was automatically disabled August 12, 2025 11:16

Pull request was closed

@github-project-automation github-project-automation bot moved this from Ready For Review to Done in Foundry Aug 12, 2025
MerkleBoy pushed a commit to MerkleBoy/foundry that referenced this pull request Sep 17, 2025
…11274, foundry-rs#11246 (foundry-rs#11279)

This PR aggregates changes from the following PRs:

- foundry-rs#10038: Make Soldeer Dependency field public by @TilakMaddy
- foundry-rs#11139: upgrade to Node.js 24 by @Daulox92
- foundry-rs#11274: CI: migrate workflows to checkout v5 by @rejected-l
- foundry-rs#11246: fix: add missing semicolon in typos error message in Makefile by @dizer-ti
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants