Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

fix: removed deprecated dependency failure and used thiserror #19

Merged
merged 1 commit into from
Nov 23, 2023
Merged

fix: removed deprecated dependency failure and used thiserror #19

merged 1 commit into from
Nov 23, 2023

Conversation

MoumitaM
Copy link
Contributor

Description of the change

Removed deprecated dependency failure and used thiserror

Type of change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Related issues

Fix #1

Checklists

Development

  • Lint rules pass locally
  • The code changed/added as part of this pull request has been covered with tests
  • All tests related to the changed code pass in development

Code review

  • This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached
  • "Ready for review" label attached to the PR and reviewers mentioned in a comment
  • Changes have been reviewed by at least one other engineer
  • Issue from task tracker has a link to this pull request

@MoumitaM MoumitaM requested a review from itsdebs November 20, 2023 11:59
@MoumitaM MoumitaM self-assigned this Nov 20, 2023
@MoumitaM MoumitaM merged commit 27c52e0 into develop Nov 23, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@itsdebs itsdebs

Assignees

@MoumitaM MoumitaM

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MoumitaM @itsdebs