Skip to content

Remove snappy#5580

Merged
bthomee merged 1 commit intobthomee/updates2from
bthomee/updates3
Jul 18, 2025
Merged

Remove snappy#5580
bthomee merged 1 commit intobthomee/updates2from
bthomee/updates3

Conversation

@bthomee
Copy link
Copy Markdown
Collaborator

@bthomee bthomee commented Jul 18, 2025

High Level Overview of Change

This PR removes the Snappy dependency as it is not referenced anywhere.

Context of Change

Cleaning up dependencies.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactor (non-breaking change that only restructures code)
  • Performance (increase or change in throughput and/or latency)
  • Tests (you added tests for code that already exists, or your new feature included in this PR)
  • Documentation update
  • Chore (no impact to binary, e.g. .gitignore, formatting, dropping support for older tooling)
  • Release

@bthomee bthomee added DraftRunCI Normally CI does not run on draft PRs. This opts in. and removed DraftRunCI Normally CI does not run on draft PRs. This opts in. labels Jul 18, 2025
@bthomee bthomee marked this pull request as ready for review July 18, 2025 19:42
@codecov
Copy link
Copy Markdown

codecov bot commented Jul 18, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (bthomee/updates2@6d882ef). Learn more about missing BASE report.
Report is 1 commits behind head on bthomee/updates2.

Additional details and impacted files

Impacted file tree graph

@@                Coverage Diff                 @@
##             bthomee/updates2   #5580   +/-   ##
==================================================
  Coverage                    ?   78.4%           
==================================================
  Files                       ?     816           
  Lines                       ?   71709           
  Branches                    ?    8598           
==================================================
  Hits                        ?   56215           
  Misses                      ?   15494           
  Partials                    ?       0

Impacted file tree graph

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@bthomee bthomee merged commit 5e557f1 into bthomee/updates2 Jul 18, 2025
3 checks passed
@bthomee bthomee deleted the bthomee/updates3 branch July 18, 2025 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@bthomee