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

Cleaning up rewrite code #89

Merged
merged 2 commits into from
Jun 27, 2014
Merged

Conversation

jimfleming
Copy link
Contributor

Handles properties individually so we no longer need to whitelist node types.
#86 (AssignmentExpression) and #64 (rewrite cleanup)

@coveralls
Copy link

Coverage Status

Coverage increased (+19.48%) when pulling de398a3 on jimfleming:rewrite_cleanup into adc0b95 on rdio:master.

@jimfleming
Copy link
Contributor Author

^ Looking into test failures.

@coveralls
Copy link

Coverage Status

Coverage increased (+19.59%) when pulling 8572c6f on jimfleming:rewrite_cleanup into adc0b95 on rdio:master.

@brettlangdon
Copy link
Contributor

so much red and green, this looks awesome.

+1

jimfleming added a commit that referenced this pull request Jun 27, 2014
@jimfleming jimfleming merged commit 1228b57 into rdio:master Jun 27, 2014
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.

None yet

3 participants