Skip to content
This repository has been archived by the owner on Aug 24, 2022. It is now read-only.

PMM-9630 Use status, status_code only for some versions. #327

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

JiriCtvrtka
Copy link
Contributor

https://jira.percona.com/browse/PMM-9630

Build: SUBMODULES-0

  • Links to other linked pull requests (optional).

@@ -259,7 +250,7 @@ func (v *pgStatMonitorAllViewType) NewStruct() reform.Struct {

// String returns a string representation of this struct or record.
func (s pgStatMonitor) String() string {
res := make([]string, 51)
res := make([]string, 49)
Copy link
Contributor

Choose a reason for hiding this comment

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

🚫 [golangci-lint] reported by reviewdog 🐶
mnd: Magic number: 49, in detected (gomnd)

@JiriCtvrtka
Copy link
Contributor Author

2 failed tests are not related to PR. See empty PR failed on same tests here: #328

This reverts commit 22ece8f.
@JiriCtvrtka JiriCtvrtka marked this pull request as ready for review March 8, 2022 09:54
@JiriCtvrtka JiriCtvrtka requested a review from BupycHuk as a code owner March 8, 2022 09:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants