Skip to content

Add a new strategy for import resolution and rename variables#343

Merged
KyrylR merged 1 commit into
BlockstreamResearch:dev/modulesfrom
LesterEvSe:update/dependency-graph
Jun 2, 2026
Merged

Add a new strategy for import resolution and rename variables#343
KyrylR merged 1 commit into
BlockstreamResearch:dev/modulesfrom
LesterEvSe:update/dependency-graph

Conversation

@LesterEvSe
Copy link
Copy Markdown
Collaborator

@LesterEvSe LesterEvSe commented Jun 2, 2026

  • Rename Module struct to SourceModule and other variables to use more appropriate and descriptive names.
  • Update the loading strategy to align with the new Module functionality.

@LesterEvSe LesterEvSe requested a review from KyrylR June 2, 2026 08:52
@LesterEvSe LesterEvSe self-assigned this Jun 2, 2026
@LesterEvSe LesterEvSe requested a review from delta1 as a code owner June 2, 2026 08:52
@LesterEvSe LesterEvSe added the enhancement New feature or request label Jun 2, 2026
KyrylR
KyrylR approved these changes Jun 2, 2026
View reviewed changes
Copy link
Copy Markdown
Collaborator

@KyrylR KyrylR left a comment

Choose a reason for hiding this comment

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

ACK 86a0100; successfully ran local tests

@KyrylR KyrylR merged commit 86a0100 into BlockstreamResearch:dev/modules Jun 2, 2026
11 checks passed
@LesterEvSe LesterEvSe mentioned this pull request Jun 2, 2026
Draft
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@KyrylR KyrylR KyrylR approved these changes

@delta1 delta1 Awaiting requested review from delta1 delta1 is a code owner

Assignees

@LesterEvSe LesterEvSe

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@LesterEvSe @KyrylR