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(deps): remove unused openssl #2881

Merged
merged 4 commits into from
Sep 22, 2024
Merged

fix(deps): remove unused openssl #2881

merged 4 commits into from
Sep 22, 2024

Conversation

amitksingh1490
Copy link
Collaborator

@amitksingh1490 amitksingh1490 commented Sep 22, 2024

Summary:
Briefly describe the changes made in this PR.

Issue Reference(s):
Fixes #... (Replace "..." with the issue number)

Build & Testing:

  • I ran cargo test successfully.
  • I have run ./lint.sh --mode=fix to fix all linting issues raised by ./lint.sh --mode=check.

Checklist:

  • I have added relevant unit & integration tests.
  • I have updated the documentation accordingly.
  • I have performed a self-review of my code.
  • PR follows the naming convention of <type>(<optional scope>): <title>

@github-actions github-actions bot added the type: fix Iterations on existing features or infrastructure. label Sep 22, 2024
@tusharmath tusharmath enabled auto-merge (squash) September 22, 2024 07:21
@amitksingh1490
Copy link
Collaborator Author

posthog.rs has a dependency to reqwest which is using openssl this id fixed in this commit but not released yet using the dep from the commit PostHog/posthog-rs@3fa3e78

@amitksingh1490
Copy link
Collaborator Author

created an issue requesting a release PostHog/posthog-rs#12

@tusharmath tusharmath merged commit 574adba into main Sep 22, 2024
40 of 45 checks passed
@tusharmath tusharmath deleted the fix/ci-builds branch September 22, 2024 08:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: fix Iterations on existing features or infrastructure.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants