jazz_hands
jazz_hands copied to clipboard
jazz_hands for non-rails projects
This gets jazz_hands working for non-rails projects. Gems are automatically required on requiring jazz_hands.
looks like a very useful feature to have
@nixme Any thought on this?
Indeed, why cannot be merged?
+1 for at least a reply from @nixme about this pull request.
Sorry, taking a look now. I do want this functionality in master
@nixme is the response from the original merge requester necessary to get this into master?
Probably although I can use this as a blueprint. Sorry been behind on my projects. I have time this Thanksgiving holiday to look at everything and merge.
@nixme happy to help if you'd like another pair of eyes.
Can’t really remember what the reporting active support dependency was for but the attribute accessors one is just for module accessors.
Perhaps an easy way to remove the dependency would to be just to copy the files wholesale into a utils directory and change the namespace. This way the functionality doesn’t need adding into the code and the existing abstractions stay.
On 28 Nov 2014, at 00:51, yn [email protected] wrote:
@nixme https://github.com/nixme happy to help if you'd like another pair of eyes.
— Reply to this email directly or view it on GitHub https://github.com/nixme/jazz_hands/pull/13#issuecomment-64840812.
Also not every project uses active support but does it harm adding it as a development dependency for use with pry?
Managed to mess up this branch. Having trouble getting jazz_hands working on my new machine.
@nixme Could you consider merging in the 'bring-your-own-debugger' branch OR switching to conditional dependencies based on the ruby version.
Shouldn't be too hard to extract out the dependencies.
anything new here?