Skip to content

Add Dependency Management#69

Closed
LesterEvSe wants to merge 2 commits into
masterfrom
feature/imports
Closed

Add Dependency Management#69
LesterEvSe wants to merge 2 commits into
masterfrom
feature/imports

Conversation

@LesterEvSe

@LesterEvSe LesterEvSe commented Apr 30, 2026

Copy link
Copy Markdown
Collaborator
  • This PR suggests a bug fix and I've added the necessary tests.
  • This PR introduces a new feature and I've discussed the update in an Issue or with the team.
  • This PR is just a minor change like a typo fix.

@LesterEvSe LesterEvSe self-assigned this Apr 30, 2026
@LesterEvSe LesterEvSe added the enhancement New feature or request label Apr 30, 2026
@LesterEvSe LesterEvSe linked an issue Apr 30, 2026 that may be closed by this pull request
pub fn new(arguments: impl ArgumentsTrait + 'static) -> Self {
Self {
program: Program::new(Self::SOURCE, Box::new(arguments)),
program: Program::new_multi(Self::SOURCE_PATH, Self::SOURCE, Self::DEPENDENCIES, Box::new(arguments)),

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

So will we always use the multi thing?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Yes, because this is an extension of the existing functionality. Single-file Simplicity programs can be run with an empty DependencyMap struct

@Arvolear Arvolear changed the base branch from master to dev April 30, 2026 13:43
}

#[derive(Debug, Clone, Default, Serialize, Deserialize)]
pub struct JsonRemapping {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Let's move this to a separate file.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Ok

Base automatically changed from dev to master May 12, 2026 19:06
@LesterEvSe LesterEvSe changed the base branch from master to dev June 2, 2026 15:01
Base automatically changed from dev to master June 9, 2026 11:57
@LesterEvSe LesterEvSe mentioned this pull request Jun 11, 2026
3 tasks
@LesterEvSe LesterEvSe added the duplicate This issue or pull request already exists label Jun 11, 2026
@LesterEvSe

Copy link
Copy Markdown
Collaborator Author

Closing in favor of #95

@LesterEvSe LesterEvSe closed this Jun 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

duplicate This issue or pull request already exists enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dependencies management

2 participants