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

Concurrency checks #4

Merged
merged 3 commits into from
Jun 15, 2024
Merged

Concurrency checks #4

merged 3 commits into from
Jun 15, 2024

Conversation

lucmq
Copy link
Owner

@lucmq lucmq commented Jun 15, 2024

No description provided.

@lucmq lucmq self-assigned this Jun 15, 2024
@deepsource-io deepsource.io
Copy link

deepsource-io bot commented Jun 15, 2024 •
edited
Loading

Here's the code health analysis summary for commits d9f6c6a..f3be7ef. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource Go LogoGo✅ SuccessView Check ↗
DeepSource Test coverage LogoTest coverage✅ SuccessView Check ↗

Code Coverage Report

MetricAggregateGo
Composite Coverage96.3% (up 0.3% from main)96.3% (up 0.3% from main)
Line Coverage96.3% (up 0.3% from main)96.3% (up 0.3% from main)
💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@lucmq
Fix range variables
f3be7ef 
Range variables were being reused in some loops in the tests
@lucmq lucmq merged commit eb6d797 into main Jun 15, 2024
5 checks passed
@lucmq lucmq deleted the concurrency-checks branch June 15, 2024 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@lucmq lucmq

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@lucmq