Advice logic in string #13

Closed
gcao opened this Issue Dec 31, 2012 · 5 comments

Projects

None yet

2 participants

@gcao
Owner

after :initialize, "@t = Test.new"
after :test, "@t.do_something"

@mensfeld
Collaborator

isn't it already in the aspector?

@gcao
Owner

It's done but I think it should probably be removed. It's an experimental feature.

@mensfeld
Collaborator

@gcao yes but on the other hand its a feature that's there and removing it would break the API - someone might use it

@gcao
Owner

If we decide to remove it, we can do it in steps and provide warnings first. Aspector is still in pre-1.0 time, even though it might be stable enough for production use. We can make API changes if it's deemed important.

That being said, it's still to be discussed whether we should remove it. Putting logic in string does not bring anything unique except that no method is created.

@mensfeld
Collaborator

Yeah I agree - there's enough better ways to do stuff than to put it in a string ;)

I will close this issue since it refers to this feature being "up" not to remove it

@mensfeld mensfeld closed this Jul 7, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment