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

refactor: cmd/phlare/main.go #566

Merged
merged 4 commits into from
Mar 9, 2023
Merged

Conversation

AkshayAwate
Copy link
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
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

@Rustin170506
Copy link
Contributor

Rustin170506 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
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

cmd/phlare/main.go Outdated Show resolved Hide resolved
@AkshayAwate AkshayAwate requested review from simonswine and cyriltovena 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