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

Migrates http client to reqwest #248

Merged
merged 2 commits into from
Dec 24, 2024
Merged

Migrates http client to reqwest #248

merged 2 commits into from
Dec 24, 2024

Conversation

ubiratansoares
Copy link
Member

@ubiratansoares ubiratansoares commented Dec 24, 2024

Description

Describe your changes in detail
Why is this change required? What problem does it solve?
If it fixes an open issue, please put a link to the issue here.

Self-explained

Types of changes

What types of changes does your code introduce? Put an x in all the boxes that apply:

  • House cleaning
  • Bug fix
  • Security fix
  • Enhancement
  • Breaking change
  • New feature
  • New release
  • Documentation

Additional details

Please, list here some additional details we should be aware of when reviewing your PR.

N/A

@ubiratansoares ubiratansoares self-assigned this Dec 24, 2024
Copy link

deepsource-io bot commented Dec 24, 2024

Here's the code health analysis summary for commits f2a838f..9d79053. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource Rust LogoRust✅ Success
❗ 1 occurence introduced
🎯 1 occurence resolved
View Check ↗
DeepSource Shell LogoShell✅ SuccessView Check ↗

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

@ubiratansoares ubiratansoares marked this pull request as ready for review December 24, 2024 14:13
@mergify mergify bot merged commit 9dc4d96 into main Dec 24, 2024
9 checks passed
@mergify mergify bot deleted the ufs/reqwest-migration branch December 24, 2024 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant