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

added changelog #76

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

added changelog #76

wants to merge 5 commits into from

Conversation

johnstonematt
Copy link
Contributor

Currently it's just a write up for v3. Will pretty much copy-paste this into the release notes, but with PR links added.

Copy link
Contributor

@SEJeff SEJeff left a comment

Choose a reason for hiding this comment

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

Would you be ok with renaming this file to CHANGELOG.md in the root which is more what the keep a changelog style is that many OSS projects follow.

Examples:

Also, the queen's english? Really? Bloody hell, my 🇺🇸 brain can not comprehend this!

update-log/v3.0.0.md Outdated Show resolved Hide resolved
update-log/v3.0.0.md Outdated Show resolved Hide resolved
@johnstonematt johnstonematt changed the title added update-log added changelog Nov 15, 2024

* `solana_account_balance` (<u><strong>[@johnstonematt](https://github.com/johnstonematt)</strong></u>)
* `solana_node_is_healthy` (<u><strong>[@GranderStark](https://github.com/GranderStark)</strong></u>)
* `solana_nude_num_slots_behind` (<u><strong>[@GranderStark](https://github.com/GranderStark)</strong></u>)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* `solana_nude_num_slots_behind` (<u><strong>[@GranderStark](https://github.com/GranderStark)</strong></u>)
* `solana_node_num_slots_behind` (<u><strong>[@GranderStark](https://github.com/GranderStark)</strong></u>)

😆

The new `solana-exporter` (renamed from `solana_exporter`) contains many new metrics, standardised naming conventions
and more configurability.

## What's Changed
Copy link
Contributor

Choose a reason for hiding this comment

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

Might be worth explicitly calling out some of the new features under this section without a subsection

  • fee reward tracking
  • easy healthy / num_slots_behind stats
  • tracking of active stake per specified validator
  • balance monitoring for any arbitrary address/es
  • block size tracking to monitor for blocks without user transactions from bad host or relayer configuration

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