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.

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

fix(uad_lists): add missing quotes #567

Merged
merged 1 commit into from
Jul 28, 2024
Merged

fix(uad_lists): add missing quotes #567

merged 1 commit into from
Jul 28, 2024

Conversation

AnonymousWP
Copy link
Member

@AnonymousWP AnonymousWP commented Jul 28, 2024

As this is a critical hot-fix, I've temporarily enabled bypassing for administrators so we do not need to wait for approval from another team member.

@AnonymousWP AnonymousWP added the bug Something isn't working label Jul 28, 2024
Copy link
Contributor

deepsource-io bot commented Jul 28, 2024

Here's the code health analysis summary for commits 4b5a079..9cad1db. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource Rust LogoRust✅ SuccessView Check ↗
DeepSource Test coverage LogoTest coverage✅ SuccessView Check ↗

Code Coverage Report

MetricAggregateRust
Branch Coverage100%0%
Composite Coverage1.9%1.9%
Line Coverage1.9%1.9%

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@AnonymousWP AnonymousWP merged commit bd486ea into main Jul 28, 2024
21 checks passed
@AnonymousWP AnonymousWP deleted the fix-broken-list branch July 28, 2024 13:26
@Rudxain
Copy link
Member

Rudxain commented Jul 29, 2024

Found the culprit of the regression: #566

@AnonymousWP
Copy link
Member Author

Found the culprit of the regression: #566

Yeah it is. Was quite obvious as it failed after I merged that PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants