Nima H. Mood

Way of calling a native Ruby less? [lib/watson/printer.rb]

1

**filename** : lib/watson/printer.rb **line #** : 131 **tag** : review **md5** : bd28c51d7b384df0b16b241866680537 ``` # [review] - Way of calling a native Ruby less? system("less #{ @config.tmp_file }") debug_print "File...

**filename** : lib/watson/remote.rb **line #** : 114 **tag** : review **md5** : bef78382fa6b9e5bcb435fac0a62189c ``` # [review] - Returning hash of json + response the right thing to do? # return...

**filename** : lib/watson/remote.rb **line #** : 109 **tag** : fix **md5** : b547385b20a5543b644943e810e1af76 ``` # [fix] - Not sure if 401 is the only code that gives nonparseable body? #...

**filename** : lib/watson/remote.rb **line #** : 88 **tag** : review **md5** : 88fbf45c55d9c0d76128faaf1de92dc8 ``` # [review] - Add :data_format to use set_form_data vs json body? # For now, use Hash...

**filename** : lib/watson/remote.rb **line #** : 64 **tag** : review **md5** : 663b8e2b89d43c517ffed3360f37b4d7 ``` # [review] - Not sure if to fail with no method or default to GET? case...

**filename** : lib/watson/remote.rb **line #** : 37 **tag** : review **md5** : 2a8d4b8ce99f23041e1318f7f87ef236 ``` # [review] - Not sure if this is the best/proper way to do things but it...

**filename** : lib/watson/remote.rb **line #** : 36 **tag** : review **md5** : 63e332a00c12dd59c26b4036206113ca ``` # [review] - Don't use DEBUG inside Remote class but pull from calling method's class? #...

**filename** : lib/watson/printer.rb **line #** : 245 **tag** : review **md5** : 2882c6fee92fb6ae2702ed8e058b70f2 ``` # [review] - Better way to ignore tags through structure (hash) data # Maybe have individual...

**filename** : lib/watson/printer.rb **line #** : 240 **tag** : review **md5** : 343388179dd367286519f7c4118d6200 ``` # [review] - Should the tag structure be self contained in the hash # Or is...

**filename** : lib/watson/printer.rb **line #** : 26 **tag** : review **md5** : 3727c6b5ea8b49483d4e32af0c5f168e ``` # [review] - No real setup in initialize method, combine it and run method? # Include...