Skip to content

Conversation

yash37158
Copy link
Contributor

@yash37158 yash37158 commented Feb 7, 2025

Notes for Reviewers
Added a reusable utility function for resource detail formatting:

  • New convertToReadableUnit function that converts bytes to human-readable units (KiB/MiB/GiB)
  • Properly exported through the component library's index files
  • This changes ensures consistency on all the repos

Extension of PR number 13486

Signed commits

  • Yes, I signed my commits.

@yash37158
Copy link
Contributor Author

@amitamrutiya ^^

@amitamrutiya
Copy link
Contributor

Lint Failing @yash37158

Signed-off-by: yash37158 <[email protected]>
@yash37158
Copy link
Contributor Author

Hey @amitamrutiya I have fixed the linting issue still check is failing

Screenshot 2025-02-10 at 12 11 25 AM

@amitamrutiya
Copy link
Contributor

It is really weird. why this is failing only on your PR?

@amitamrutiya
Copy link
Contributor

try running npm run format:write and npm run lint:fix

@yash37158
Copy link
Contributor Author

@amitamrutiya I have pushed the fix

@amitamrutiya amitamrutiya merged commit 6362567 into layer5io:master Feb 10, 2025
5 checks passed
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.

2 participants