Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(plugins): make rust plugin stage compatible with webpack #7942

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

pawel-twardziak
Copy link

Summary

see #7862

Checklist

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

@CLAassistant
Copy link

CLAassistant commented Sep 20, 2024

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot added the release: bug fix release: bug related release(mr only) label Sep 20, 2024
Copy link

netlify bot commented Sep 20, 2024

Deploy Preview for rspack ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit e4afa28
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/66f4332c23845b00088497e2
😎 Deploy Preview https://deploy-preview-7942--rspack.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@pawel-twardziak pawel-twardziak force-pushed the fix/rust-plugin-stage-not-compatible-with-webpack branch 2 times, most recently from 6fa723a to 149dd33 Compare September 23, 2024 20:24
@hardfist
Copy link
Contributor

can you add test about it, try pass Infinity to stage and see whether it works

@pawel-twardziak
Copy link
Author

@hardfist sure, will do and let you know

@pawel-twardziak pawel-twardziak force-pushed the fix/rust-plugin-stage-not-compatible-with-webpack branch 2 times, most recently from 4a7f682 to d55e2a9 Compare September 24, 2024 17:18
@@ -1419,7 +1419,7 @@ impl NormalModuleFactoryResolve for NormalModuleFactoryResolveTap {
}

fn stage(&self) -> i32 {
Copy link
Contributor

Choose a reason for hiding this comment

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

should change to f64 too

Copy link
Author

Choose a reason for hiding this comment

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

yes, but there ara hard implications when I change it :) Let me show you @hardfist

Copy link
Author

Choose a reason for hiding this comment

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

@hardfist When I change it, I see this while bouilding:
obraz
And it seems I need to align way more code...

@pawel-twardziak pawel-twardziak force-pushed the fix/rust-plugin-stage-not-compatible-with-webpack branch from b995597 to e4afa28 Compare September 25, 2024 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release: bug fix release: bug related release(mr only)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants