Skip to content
This repository has been archived by the owner on Aug 23, 2020. It is now read-only.

Add method body -> super mutation #8

Merged
merged 1 commit into from Jan 28, 2017
Merged

Add method body -> super mutation #8

merged 1 commit into from Jan 28, 2017

Conversation

dgollahon
Copy link
Owner

dgollahon
dgollahon commented Jan 28, 2017
View reviewed changes
meta/def.rb
@@ -64,6 +68,9 @@

# Failing body
mutation 'def a; raise; end'

# Superclass implementation
Copy link
Owner Author

Choose a reason for hiding this comment

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

I added comments where it felt most consistent to do so, but it's kind of inconsistent throughout these to begin with so idk what's best.

@dgollahon
Copy link
Owner Author

Note: I considered adding super() as well, but I figure that it'd mutate to that if it ever mattered in a second run.

@dgollahon dgollahon force-pushed the feature/method_body_to_super_mutation branch from 3f95d9f to 12e19cb Compare January 28, 2017 10:46
@backus backus merged commit dc712c5 into master Jan 28, 2017
@backus backus deleted the feature/method_body_to_super_mutation branch January 28, 2017 22:21
@backus
Copy link
Collaborator

backus commented Jan 28, 2017

Looks good

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@dgollahon @backus