Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: display shard size #94

Merged
merged 2 commits into from
Sep 8, 2023
Merged

feat: display shard size #94

merged 2 commits into from
Sep 8, 2023

Conversation

alanshaw
Copy link
Member

@alanshaw alanshaw commented Sep 8, 2023

This PR simply displays a nice human readable value for the shard size next to each shard:

Screenshot 2023-09-08 at 16 07 32

@alanshaw alanshaw requested a review from olizilla September 8, 2023 15:18
@olizilla
Copy link
Contributor

olizilla commented Sep 8, 2023

E_TOO_MANY_BRACES. and also make it look nice plz, like yer ipld-explorer thinger with dim colours for the sizes.

@alanshaw
Copy link
Member Author

alanshaw commented Sep 8, 2023

I make colours
Screenshot 2023-09-08 at 16 37 06

@alanshaw alanshaw merged commit 59e22cb into main Sep 8, 2023
@alanshaw alanshaw deleted the feat/display-shard-size branch September 8, 2023 15:41
alanshaw pushed a commit that referenced this pull request Sep 8, 2023
🤖 I have created a release *beep* *boop*
---


##
[4.2.0](v4.1.2...v4.2.0)
(2023-09-08)


### Features

* display shard size
([#94](#94))
([59e22cb](59e22cb))


### Bug Fixes

* don't error when email address contains +
([#90](#90))
([8240ba5](8240ba5))
* readme
([22a9312](22a9312))
* readme node version and link to space explainer
([#92](#92))
([b63b220](b63b220))
* various linting issues
([#93](#93))
([f829c42](f829c42))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants