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

2024 12 13 int test cleanup #39

Merged
merged 13 commits into from
Jan 9, 2025
Merged

2024 12 13 int test cleanup #39

merged 13 commits into from
Jan 9, 2025

Conversation

jbesraa
Copy link
Owner

@jbesraa jbesraa commented Dec 13, 2024

No description provided.

@jbesraa jbesraa force-pushed the 2024-12-13-int-test-cleanup branch 2 times, most recently from 1f80f31 to 0a81c0e Compare December 18, 2024 11:42
@jbesraa jbesraa force-pushed the 2024-12-13-int-test-cleanup branch from 0a81c0e to 1a1a43d Compare December 30, 2024 10:47
@jbesraa jbesraa force-pushed the 2024-12-13-int-test-cleanup branch 3 times, most recently from 9b3ce59 to 1814d1d Compare January 6, 2025 13:36
@jbesraa jbesraa force-pushed the 2024-12-13-int-test-cleanup branch 3 times, most recently from 1cefc7c to e55db1d Compare January 9, 2025 12:30
plebhash and others added 6 commits January 9, 2025 14:32
it's important to have a descriptive name for when this is published to crates.io
into the following modules:
- lib: with general purpose functions (e.g.: starters)
- sniffer
- template_provider
this struct is redundant and no other roles follow this pattern
we can do the initialization inside `start_pool`
the `SocketAddr` is not a client, but the socket where we will listen to
also, `listner` is a typo
the original naming was confusing copypasta
..to hold utility functions used internaly only.
@jbesraa jbesraa force-pushed the 2024-12-13-int-test-cleanup branch from e55db1d to d7e6f3b Compare January 9, 2025 12:34
@jbesraa jbesraa merged commit a053dc7 into main Jan 9, 2025
35 checks passed
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.

2 participants