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

Update, Sort, Color code and table format to make it more readable. #1546

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AdrianGeorgeM
Copy link

@AdrianGeorgeM AdrianGeorgeM commented May 4, 2023

💭 Why do we need this change

  • Optimize and update the process for rarity.js tables to make it more efficient.
    Screenshot 2023-05-04 at 22 58 08

🤓 Implementation details

  1. Use a table format: Instead of just printing out the data as text, you can use a table format to make it more readable.
    There are several libraries you can use for this, such as cli-table

  2. Sort the data: You can sort the data by rarity so that the rare traits are at the top of the table.
    This will make it easier to see which traits are the most valuable.

  3. Color code the data: You can use colour coding to highlight the rarest traits.
    For example, you can use green for the most common traits, yellow for moderately rare traits, and red for the most rare traits.

🛠️ How to test.

node index.js 
yarn add cli-table  
npm run rarity 

❗ Gotcha's

Video demonstration
https://recordit.co/hHaj4pM5Dj

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.

1 participant