console.table icon indicating copy to clipboard operation
console.table copied to clipboard

Support for nested arrays - fixes #10

Open fearphage opened this issue 8 years ago • 2 comments

I took a stab at fixing this. In the process, I did a lot of refactoring.

If you don't like it, I can separate out the parts.

Changes:

  • feature: ability to include library without registration (manual: require('console.table/lib') vs automatic: require('console.table'))
  • feature: ability to change the console method from console.log
  • feature: option to handle nested arrays

I also cleaned up the tests a lot and removed a lot of redundancies. Generally it's best to have people opt-in to globabl changes like this (see: babel for instace), but that would be a breaking change so I didn't do.

Let me know your thoughts.

fearphage avatar May 21 '16 22:05 fearphage

Hi! I appreciate what you are doing, seems you need this module more than I do right now. I have added you as a collaborator, so you can merge your own PR, and please do not hesitate to change any other thing. The semantic release should take care of NPM publish as log as you mark your commit using feat(), fix() semantic-release rule.

bahmutov avatar May 23 '16 01:05 bahmutov

If you resolve the conflicts and name commits (or even empty commit on top) with BREAKING CHANGE in the body, it would be great, then semantic release would pick it up

bahmutov avatar Dec 21 '16 15:12 bahmutov