Skip to content

Add DexPaprika to Tools#15

Open
donbagger wants to merge 1 commit into
Helmi:masterfrom
donbagger:add-dexpaprika
Open

Add DexPaprika to Tools#15
donbagger wants to merge 1 commit into
Helmi:masterfrom
donbagger:add-dexpaprika

Conversation

@donbagger

Copy link
Copy Markdown

Adds DexPaprika to the Tools section — DEX analytics platform covering 34 blockchains with pools, tokens, OHLCV charts, trade history, and real-time streaming.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds DexPaprika to the tools section of the README. The review feedback suggests reordering the new entry to maintain the alphabetical organization of the list and provides a suggestion to improve the readability of the description by combining fragmented phrases.

Comment thread README.md
- [Ghostfolio](https://ghostfol.io/): Open source wealth management software to track cryptocurrency holdings across multiple platforms
- [Mobula UI](https://github.com/MobulaFi/mobula-ui): Open-source coin & portfolio tracking platform
- [RP2](https://github.com/eprbell/rp2): Privacy-focused, free, open-source crypto tax calculator supporting multiple countries
- [DexPaprika](https://dexpaprika.com/) - DEX analytics across 34 blockchains. Pools, tokens, OHLCV charts, trade history, and [real-time streaming](https://docs.dexpaprika.com/streaming).

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The 'Tools' section appears to follow an alphabetical order for its initial entries (Coiniverse, Ghostfolio, Mobula UI, RP2). To maintain this organization and help users find entries more easily, 'DexPaprika' should be moved to follow 'Coiniverse' and precede 'Ghostfolio'.

Comment thread README.md
- [Ghostfolio](https://ghostfol.io/): Open source wealth management software to track cryptocurrency holdings across multiple platforms
- [Mobula UI](https://github.com/MobulaFi/mobula-ui): Open-source coin & portfolio tracking platform
- [RP2](https://github.com/eprbell/rp2): Privacy-focused, free, open-source crypto tax calculator supporting multiple countries
- [DexPaprika](https://dexpaprika.com/) - DEX analytics across 34 blockchains. Pools, tokens, OHLCV charts, trade history, and [real-time streaming](https://docs.dexpaprika.com/streaming).

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The description is currently split into two fragments, which makes it feel disjointed. Combining them into a single cohesive phrase would improve readability and better align with the style of other entries in the list. The version used in your PR description is a great improvement.

Suggested change
- [DexPaprika](https://dexpaprika.com/) - DEX analytics across 34 blockchains. Pools, tokens, OHLCV charts, trade history, and [real-time streaming](https://docs.dexpaprika.com/streaming).
- [DexPaprika](https://dexpaprika.com/) - DEX analytics platform covering 34 blockchains with pools, tokens, OHLCV charts, trade history, and [real-time streaming](https://docs.dexpaprika.com/streaming).
References
  1. Descriptions should be concise and well-structured as per the contribution guidelines.

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