Skip to content
New issue

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

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

Already on GitHub? # to your account

Prepare a new release. #44

Merged
merged 5 commits into from
Jun 2, 2020
Merged

Prepare a new release. #44

merged 5 commits into from
Jun 2, 2020

Conversation

AlekSi
Copy link

@AlekSi AlekSi commented Jun 1, 2020

No description provided.

@AlekSi AlekSi requested a review from BupycHuk June 1, 2020 13:25
@AlekSi AlekSi self-assigned this Jun 1, 2020
CHANGELOG.md Outdated
- `disable_basic_metrics` and `disable_enhanced_metrics` configuration options.

### Changed
- Major changes in `node_exporter`-like metrics.
Copy link
Member

Choose a reason for hiding this comment

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

I think it's good to add a link what's changed in these metrics(naming and e.t.c.)

Copy link
Author

Choose a reason for hiding this comment

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

Sure, what link should be used there?

Copy link
Member

Choose a reason for hiding this comment

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

Maybe description from this PR #33?

@AlekSi AlekSi requested a review from BupycHuk June 1, 2020 13:48
CHANGELOG.md Outdated
- `disable_basic_metrics` and `disable_enhanced_metrics` configuration options.

### Changed
- Major changes in `node_exporter`-like metrics. See [here](https://github.com/percona/rds_exporter#metrics).
Copy link
Author

Choose a reason for hiding this comment

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

I think that's the best we can do here.

@codecov
Copy link

codecov bot commented Jun 1, 2020

Codecov Report

Merging #44 into master will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #44   +/-   ##
=======================================
  Coverage   81.91%   81.91%           
=======================================
  Files           9        9           
  Lines         752      752           
=======================================
  Hits          616      616           
  Misses        118      118           
  Partials       18       18           
Impacted Files Coverage Δ
client/transport.go 87.87% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 81809c7...ce5de1e. Read the comment docs.

@AlekSi AlekSi merged commit 8ce49ef into master Jun 2, 2020
@AlekSi AlekSi deleted the prepare-release branch June 2, 2020 10:27
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants