Skip to content

feat(core): use dyn output_filesystem#7817

Merged
h-a-n-a merged 4 commits intomainfrom
use-dyn-output-filesystem
Sep 9, 2024
Merged

feat(core): use dyn output_filesystem#7817
h-a-n-a merged 4 commits intomainfrom
use-dyn-output-filesystem

Conversation

@hardfist
Copy link
Copy Markdown
Contributor

@hardfist hardfist commented Sep 6, 2024 •
edited
Loading

Summary

Use dynamic dispatch output_filesystem so can support dynamic switch output_filesystem in the future

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 Sep 6, 2024
@netlify
Copy link
Copy Markdown

netlify bot commented Sep 6, 2024 •
edited
Loading

✅ Deploy Preview for rspack canceled.

Name Link
🔨 Latest commit dd96b3f
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/66deb639408ba700084c2abf

@github-actions github-actions bot added the team The issue/pr is created by the member of Rspack. label Sep 6, 2024
@hardfist hardfist requested a review from h-a-n-a September 6, 2024 02:55
h-a-n-a
h-a-n-a previously approved these changes Sep 6, 2024
View reviewed changes
@hardfist hardfist changed the title feat(chore): use dyn output_filesystem feat(core): use dyn output_filesystem Sep 6, 2024
@hardfist hardfist requested review from h-a-n-a September 6, 2024 03:55
@h-a-n-a h-a-n-a enabled auto-merge (squash) September 6, 2024 03:56
@h-a-n-a h-a-n-a disabled auto-merge September 6, 2024 03:57
@h-a-n-a h-a-n-a enabled auto-merge (squash) September 6, 2024 03:57
h-a-n-a
h-a-n-a previously approved these changes Sep 6, 2024
View reviewed changes
@hardfist hardfist force-pushed the use-dyn-output-filesystem branch from 770a9a7 to b3d04fa Compare September 9, 2024 02:11
@h-a-n-a h-a-n-a merged commit aca19d4 into main Sep 9, 2024
@h-a-n-a h-a-n-a deleted the use-dyn-output-filesystem branch September 9, 2024 09:02
@renovate renovate bot mentioned this pull request Sep 26, 2024
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@LingyuCoder LingyuCoder LingyuCoder approved these changes

@h-a-n-a h-a-n-a Awaiting requested review from h-a-n-a

Assignees

@hardfist hardfist

Labels

release: feature release: feature related release(mr only) team The issue/pr is created by the member of Rspack.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@hardfist @LingyuCoder @h-a-n-a