Skip to content

feat: support compilation.modules[i].dependencies#8050

Merged
SyMind merged 1 commit intoweb-infra-dev:mainfrom
inottn:feat/module
Nov 5, 2024
Merged

feat: support compilation.modules[i].dependencies#8050
SyMind merged 1 commit intoweb-infra-dev:mainfrom
inottn:feat/module

Conversation

@inottn
Copy link
Copy Markdown
Collaborator

@inottn inottn commented Oct 3, 2024

Summary

https://discord.com/channels/977448667919286283/1076017776583516220/1291085434042060931

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@github-actions github-actions bot added the release: feature release: feature related release(mr only) label Oct 3, 2024
@netlify
Copy link
Copy Markdown

netlify bot commented Oct 3, 2024

Deploy Preview for rspack canceled.

Built without sensitive environment variables

Name Link
🔨 Latest commit b0eba69
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/67264dab362b69000858a22e

@chenjiahan chenjiahan requested a review from hardfist October 3, 2024 09:32
@hardfist
Copy link
Copy Markdown
Contributor

hardfist commented Oct 3, 2024

we need discuss whether need to support this

@chenjiahan
Copy link
Copy Markdown
Member

@hardfist any concerns?

@hardfist
Copy link
Copy Markdown
Contributor

@hardfist any concerns?

need to discuss how to efficent deal with module graph and chunk graph,will raise a RFC discuss about this in the future

@hardfist hardfist added the to be discussed Rspack team would discuss these issues per week label Oct 10, 2024
@SyMind
Copy link
Copy Markdown
Member

SyMind commented Nov 1, 2024

I will track this PR and benchmark the performance of traversing the dependencies.

@SyMind SyMind merged commit 2c8e351 into web-infra-dev:main Nov 5, 2024
@SyMind SyMind mentioned this pull request Nov 5, 2024
2 tasks
@inottn inottn deleted the feat/module branch November 5, 2024 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release: feature release: feature related release(mr only) to be discussed Rspack team would discuss these issues per week

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants