-
Notifications
You must be signed in to change notification settings - Fork 400
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
feat: implement some expr side effect detector #807
Conversation
✅ Deploy Preview for rolldown-rs canceled.
|
CodSpeed Performance ReportMerging #807 will not alter performanceComparing Summary
|
a73f5b6
to
931cdd4
Compare
@hyf0 The code for By the way, I'd like to contribute more to Rolldown. Currently, I usually look for TODO comments in the code and then implement them. However, some TODOs seem quite challenging to implement. Could you give me some advice on what aspects of Rolldown I should focus on to contribute more effectively? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
06c7054
to
6dbba6d
Compare
Description
I have enhanced some of the code for side-effect detection in expressions, but there are still parts that I'm currently unsure how to implement.