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

use provide/inject #168

Merged
merged 1 commit into from
Feb 21, 2024
Merged

use provide/inject #168

merged 1 commit into from
Feb 21, 2024

Conversation

ri-pandey
Copy link
Contributor

Description

Implementing the feedback from #167.

Related Issue(s)

Closes #158

If applicable, please reference the issue(s) that this PR addresses. If the PR does not address any specific issue, you can remove this section.

Changes Made

List the main changes made in this PR. Be as specific as possible.

  • Code refactored
    Checklist

Before submitting this PR, please make sure that:

  • Your code passes linting and coding style checks.
  • Documentation has been updated to reflect the changes.
  • You have reviewed your own code and resolved any merge conflicts.
  • You have requested a review from at least one team member.
  • Any relevant issue(s) have been linked to this PR.

@ri-pandey ri-pandey self-assigned this Feb 21, 2024
@ri-pandey ri-pandey merged commit 7d8898b into dev Feb 21, 2024
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.

None yet

1 participant