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

PMM-9634 Timestamp bucket_start_time for newer versions.#329

Open
JiriCtvrtka wants to merge 6 commits intoPMM-9630-status-code-usagefrom
PMM-9634-timestamp
Open

PMM-9634 Timestamp bucket_start_time for newer versions.#329
JiriCtvrtka wants to merge 6 commits intoPMM-9630-status-code-usagefrom
PMM-9634-timestamp

Conversation

@JiriCtvrtka
Copy link
Copy Markdown
Contributor

PMM-9634

Build: SUBMODULES-0

  • Links to other linked pull requests (optional).

Branch is based on 9630.

}

func NewPgStatMonitorStructs(v pgStatMonitorVersion) (*pgStatMonitor, reform.View) {
func NewPgStatMonitorStructs(v pgStatMonitorVersion, p pgStatMonitorPrerelease) (*pgStatMonitor, reform.View) {
Copy link
Copy Markdown
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 🐶
Function 'NewPgStatMonitorStructs' is too long (114 > 60) (funlen)

@JiriCtvrtka
Copy link
Copy Markdown
Contributor Author

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

@JiriCtvrtka JiriCtvrtka marked this pull request as ready for review March 10, 2022 11:57
@JiriCtvrtka JiriCtvrtka requested a review from BupycHuk as a code owner March 10, 2022 11:57
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.

3 participants