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.

Sign up for GitHub

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

Jump to bottom

[WIP] Remove default new for modules #3887

Closed
wants to merge 1 commit into from
Closed

[WIP] Remove default new for modules #3887

wants to merge 1 commit into from

Conversation

david50407
Copy link
Contributor

Fixed #3885 .

Now will throw the correct message of executing SomeModule.new:

module Foo; end

Foo.new # undefined method 'new' for Foo:Module

@david50407 david50407 force-pushed the no_default_new_in_module branch from 97146b3 to 441c757 Compare January 13, 2017 18:40
@david50407
Copy link
Contributor Author

Oops, I missed somthing

@david50407 david50407 closed this Jan 13, 2017
@david50407 david50407 changed the title Remove default new for modules [WIP] Remove default new for modules Jan 13, 2017
@spalladino spalladino added pr:wip and removed pr:wip labels Jan 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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.
2 participants
@david50407 @spalladino