Skip to content

Conversation

alexeyr-ci2
Copy link
Collaborator

@alexeyr-ci2 alexeyr-ci2 commented Apr 11, 2025

Summary

Make imports from react-on-rails tree-shakable to improve bundle size.

This is a breaking change; do not merge until we start working on 16.0.0 (and recommended after #1710)!

Pull Request checklist

  • Add/update test to cover these changes
  • Update documentation
  • Update CHANGELOG file
  • TODO: specify sideEffects: false in package.json once everything works otherwise and verify it doesn't break anything. Check bundle size benefits.

This change is Reviewable

Copy link
Contributor

coderabbitai bot commented Apr 11, 2025

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch alexeyr/improve-tree-shakability

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@alexeyr-ci alexeyr-ci force-pushed the alexeyr/improve-tree-shakability branch 4 times, most recently from 58c9ad8 to 9878c51 Compare April 22, 2025 08:03
@justin808
Copy link
Member

@alexeyr-ci2 What's the point of this PR?

@alexeyr-ci alexeyr-ci force-pushed the alexeyr/improve-tree-shakability branch 2 times, most recently from 44b60c5 to 49b623c Compare September 15, 2025 12:43
@alexeyr-ci2
Copy link
Collaborator Author

@justin808 Added summary.

@alexeyr-ci alexeyr-ci force-pushed the alexeyr/improve-tree-shakability branch from 49b623c to 25eece5 Compare September 15, 2025 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants