Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Fix lint error in service.go file #1933

Merged
merged 1 commit into from
Oct 5, 2023
Merged

Fix lint error in service.go file #1933

merged 1 commit into from
Oct 5, 2023

Conversation

misraved
Copy link
Contributor

@misraved misraved commented Oct 4, 2023

Integration test logs

Logs 
Add passing integration test logs here

Example query results

Results 
Add example SQL query results here (please include the input queries as well)

@misraved misraved self-assigned this Oct 4, 2023
@misraved misraved changed the title Fixc lint error in service.go file Fix lint error in service.go file Oct 4, 2023
@misraved misraved requested a review from graza-io October 4, 2023 14:40
graza-io
graza-io approved these changes Oct 4, 2023
View reviewed changes
Copy link
Contributor

@graza-io graza-io left a comment

Choose a reason for hiding this comment

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

This looks like the recommended approach.

@cbruno10 cbruno10 merged commit 518407d into main Oct 5, 2023
1 check passed
@cbruno10 cbruno10 deleted the fix-lint-error-rand branch October 5, 2023 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@graza-io graza-io graza-io approved these changes

Assignees

@misraved misraved

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@misraved @graza-io @cbruno10