Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Using namespace::autoclean inflates Moo classes to Moose #6

Closed
autarch opened this Issue Dec 25, 2013 · 2 comments

Comments

Projects
None yet
3 participants
Contributor

autarch commented Dec 25, 2013

I'd really like to see this fixed in ns::ac itself. We could probably steal code from namespace::sweep to do this.

Contributor

autarch commented Dec 25, 2013

(14:11:49) haarg: autarch: i'd recommend you avoid namespace::autoclean for Moo classes. autoclean will cause Moo to inflate the Moose meta objects for the classes, largely defeating the point of using Moo.
(14:12:00) autarch: where am I doing that?
(14:12:02) autarch: Net::Works?
(14:12:08) haarg: yeah
(14:12:14) haarg: at least it's listed in the prereqs
(14:12:16) autarch: can we fix ns::ac?
(14:12:28) haarg: well, there's namespace::sweep
(14:12:36) autarch: oh good, another parallel track of modules
(14:12:42) haarg: which basically does "fix" it
(14:12:43) autarch: because that's what we need more of!
(14:13:09) haarg: well, autoclean could be changed to basically work like sweep
(14:13:20) autarch: yeah, I think that'd make the most sense
(14:13:21) haarg: which will use the Moose/Mouse meta class if it's available
(14:13:33) autarch: the overload thing is super annoying with ns::ac too
(14:13:39) autarch: ether: what do you think?
(14:13:40) purl: I think autarch should try flossing more often!
(14:13:53) haarg: but falls back on a heuristic based on sub name, which is basically a copy of Moose's heuristic
(14:15:03) haarg: i think ether is afk for a number of days

haarg commented Jan 7, 2014

I've opened a pull request attempting to fix this: moose/namespace-autoclean#1

@oschwald oschwald closed this Jul 3, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment