fix: human readable option

[PedroMiguelSS]

This PR fixes the output of human option that can be passed to repo.stat ([options]).

Both repoSize and storageMax properties of the returned object are now according to go implementation:

• repoSize -> represented in MB
• storageMax -> represented in GB

Example:

const stats = await repo.stat({ human: true })
console.log(stats)
// Logs:
//{
//  repoPath: '/var/folders/l1/l5dg09hj7x9d6yxm5z2vsr2h0000gn/T/test-repo-for-1573141229356',
//  storageMax: '10 GB',
//  version: 7,
//  numObjects: 24,
//  repoSize: '0.07 MB'
//}

Closes ipfs/js-ipfs#1996.

[PedroMiguelSS]

Can I get a second review here, please? @achingbrain @hugomrdias @jacobheun

[jacobheun]

Released in v0.29.1

[alanshaw]

I'd like to keep formatting functionality out of ipfs core - it bloats the bundle size, makes assumptions about locale and can be easily added by applications using IPFS.

What do you think? I'd be in favour of removing this functionality altogether and pushing it into the CLI and HTTP API resources.

[PedroMiguelSS]

I got your point. It makes sense. What do you think @achingbrain @jacobheun?

[jacobheun]

What do you think? I'd be in favour of removing this functionality altogether and pushing it into the CLI and HTTP API resources.

This sounds like a good approach to me.

[achingbrain]

Me too, formatting is a UI concern.