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

chore: use more semantic naming profile #9288

Merged
merged 10 commits into from
Feb 14, 2025
Merged

chore: use more semantic naming profile #9288

merged 10 commits into from
Feb 14, 2025

Conversation

hardfist
Copy link
Contributor

@hardfist hardfist commented Feb 13, 2025

Summary

follow servo's style https://github.com/servo/servo/blob/main/Cargo.toml#L181 and use more semantic profile name

  • dev: for local development with best build time and debugInfo
  • release: for release build with best runtime performance but with long build time
  • ci: based on release but with faster compile with little runtime performance lost used in ci
  • profiling: based on release with debugInfo for profiling

Checklist

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

@github-actions github-actions bot added the team The issue/pr is created by the member of Rspack. label Feb 13, 2025
Copy link

netlify bot commented Feb 13, 2025

Deploy Preview for rspack canceled.

Name Link
🔨 Latest commit f9b12d9
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/67aec1a5258c940008f850e7

@hardfist hardfist changed the title chore: use more semantic profile chore: use more semantic naming profile Feb 14, 2025
jerrykingxyz
jerrykingxyz previously approved these changes Feb 14, 2025
@hardfist hardfist enabled auto-merge (squash) February 14, 2025 03:35
@hardfist hardfist merged commit 5458f29 into main Feb 14, 2025
30 of 31 checks passed
@hardfist hardfist deleted the yj/use-profile branch February 14, 2025 04:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team The issue/pr is created by the member of Rspack.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants