Re-add awesome_print itself

rking commented 11 years ago

pry-awesome_print currently busts paging, and besides it's not really what you always want.

Better is to be able to always toss in:

ap some_obj

So, this Issue refers to:

adding awesome_print as a dep of this gem.
require 'awesome_print' in lib/pry-full.rb
Maybe preconfigure? AwesomePrint.defaults = { :raw => true } This is big but it's kind of nice opinionation, IMO.

skull-squadron commented 11 years ago

It's better for the community if we work together, not tear other people's projects down without either feedback or credit.

https://github.com/steakknife/pry-awesome_print/issues/1

skull-squadron commented 11 years ago

Preconfigure AwesomePrint.defaults using ~/.aprc or similar method via https://github.com/steakknife/pry-awesome_print/issues/3

rking commented 11 years ago

Hi @steakknife

Sorry it came out as a tear-down.

The whole job of pry-full is to be a gateway so that people can get a ton of plugins, but not get any breakage (e.g. as they would if they installed every pry-___ gem on rubygems). When I pinpointed pry-awesomeprint as the thing that was causing paging not to work, I thought about whether I wanted it on by default or not, and that's when I decided that, for me, it's nice to have the 2 modes: default as compact, but `ap ` on the quickdraw.

I suppose I can always just .inspect if I want the plain output. I'll think about it/play with it.

Anyway, much appreciation for the fix, even with my attitude about the whole thing.*

BTW, keep in mind that every time you're using Pry, it's because you're waist-deep in some other task. I had been putting up with pagerlessness over SSH for a couple weeks before I finally stopped to bisect what the problem was, so I was grumpy. =)

rking / pry-full

Re-add awesome_print itself #3