Skip to content
This repository was archived by the owner on Jul 19, 2023. It is now read-only.

refactor: cmd/phlare/main.go#566

Merged
cyriltovena merged 4 commits intografana:mainfrom
AkshayAwate:refactor-main
Mar 9, 2023
Merged

refactor: cmd/phlare/main.go#566
cyriltovena merged 4 commits intografana:mainfrom
AkshayAwate:refactor-main

Conversation

@AkshayAwate
Copy link
Copy Markdown
Contributor

Given that identical segments of code are being employed in multiple instances, it is better to utilize the concept of functions, whereby the code is encapsulated within a named block that can be invoked at various points within the program.

Copy link
Copy Markdown
Collaborator

@cyriltovena cyriltovena left a comment

Choose a reason for hiding this comment

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

LGTM

@cyriltovena cyriltovena enabled auto-merge (squash) March 7, 2023 08:17
auto-merge was automatically disabled March 7, 2023 10:35

Head branch was pushed to by a user without write access

@0xPoe
Copy link
Copy Markdown
Contributor

0xPoe commented Mar 7, 2023

@cyriltovena I guess you need to approve the first-time contributor's first PR to run the GitHub actions.
image

@AkshayAwate AkshayAwate requested a review from cyriltovena March 7, 2023 15:18
Copy link
Copy Markdown
Collaborator

@simonswine simonswine left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread cmd/phlare/main.go Outdated
@AkshayAwate AkshayAwate requested review from cyriltovena and simonswine and removed request for cyriltovena and simonswine March 8, 2023 12:53
@cyriltovena cyriltovena enabled auto-merge (squash) March 9, 2023 10:36
@cyriltovena cyriltovena merged commit dbd80d7 into grafana:main Mar 9, 2023
simonswine pushed a commit to simonswine/pyroscope that referenced this pull request Jun 30, 2023
* refactor: cmd/phlare/main.go

* Update main.go

* refactor: cmd/phlare/main.go
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