Skip to content

Include optional dependency #222

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
Merged
merged 2 commits into from
Apr 14, 2025
Merged

Include optional dependency #222

merged 2 commits into from
Apr 14, 2025

Conversation

matheusverifiedinc
Copy link
Contributor

Summary

Include optional dependency

Changes

Include optional dependency

Testing

Checklist

  • I have performed a self-review of my code.
  • I have commented my code, particularly in hard-to-understand areas to a borderline excessive amount
  • I have made any relevant changes to the documentation, including the project readme.
  • I have run and tested the changes locally
  • If it is a core feature, I have added appropriate unit tests.
  • Any dependent changes have been merged and published in upstream projects.

@matheusverifiedinc matheusverifiedinc added the hotfix Quick, necessary fix. No review needed. label Apr 14, 2025
@matheusverifiedinc matheusverifiedinc requested a review from a team April 14, 2025 13:52
@matheusverifiedinc matheusverifiedinc self-assigned this Apr 14, 2025
@matheusverifiedinc matheusverifiedinc requested review from rsmets, leo1994, iagormoraes and gabrielborgesdm and removed request for a team April 14, 2025 13:52
@gitstream-cm gitStream.cm
Copy link

gitstream-cm bot commented Apr 14, 2025

🚨 gitStream Monthly Automation Limit Reached 🚨

Your organization has exceeded the number of pull requests allowed for automation with gitStream.
Monthly PRs automated: 701/250

To continue automating your PR workflows and unlock additional features, please contact LinearB.

@gitstream-cm gitStream.cm
Copy link

gitstream-cm bot commented Apr 14, 2025

This PR has been approved because it is labeled as a hotfix.

@gabrielborgesdm gabrielborgesdm merged commit 27a6670 into main Apr 14, 2025
7 checks passed
@gabrielborgesdm gabrielborgesdm deleted the include-optional-dependency branch April 14, 2025 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gabrielborgesdm gabrielborgesdm gabrielborgesdm approved these changes

@gitstream-cm gitstream-cm[bot] gitstream-cm[bot] approved these changes

@rsmets rsmets Awaiting requested review from rsmets rsmets was automatically assigned from VerifiedInc/core-developers

@leo1994 leo1994 Awaiting requested review from leo1994 leo1994 was automatically assigned from VerifiedInc/core-developers

@iagormoraes iagormoraes Awaiting requested review from iagormoraes iagormoraes was automatically assigned from VerifiedInc/core-developers

Assignees

@matheusverifiedinc matheusverifiedinc

Labels
1 min review hotfix Quick, necessary fix. No review needed.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@matheusverifiedinc @gabrielborgesdm