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.

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

minor improvements to flags module, fix packet handling and luxos-run #41

Open
wants to merge 16 commits into
base: main
Choose a base branch
from
Open

minor improvements to flags module, fix packet handling and luxos-run #41

wants to merge 16 commits into from

Conversation

cav71
Copy link
Contributor

@cav71 cav71 commented Sep 13, 2024 •
edited
Loading

Description

  • splitip - better support for handling ip addresses
  • misc.loadmod - support for remote files
  • luxor_run - support setup/teardown
  • luxor_run - support remove modules
  • luxor_run - progress bar
  • syncops - support for 0x00 terminated packets

Merge request checklist

  • Added the appropriate bug, enhancement and/or breaking-change tags
  • I have updated the CHANGELOG (yeah, last chance to do it).
  • My code passes all tests, linter and formatting checks:
    make check
    make test

@cav71 cav71 self-assigned this Sep 13, 2024
@cav71 cav71 added the enhancement New feature or request label Sep 13, 2024
@cav71 cav71 force-pushed the antoniocavallo/minor-cleanups branch from 878c75b to 59b326b Compare September 13, 2024 21:18
@cav71 cav71 changed the title Antoniocavallo/minor cleanups minor improvements to flags module, new version module Sep 22, 2024
@cav71 cav71 force-pushed the antoniocavallo/minor-cleanups branch from 98171f8 to dac0c7a Compare September 23, 2024 19:39
@cav71 cav71 force-pushed the antoniocavallo/minor-cleanups branch from e93d0d3 to 89f5cc4 Compare November 14, 2024 03:36
@cav71 cav71 changed the title minor improvements to flags module, new version module minor improvements to flags module, fix packet handling and luxos-run Nov 15, 2024
@cav71 cav71 marked this pull request as ready for review November 15, 2024 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dpineda64 dpineda64 Awaiting requested review from dpineda64

At least 1 approving review is required to merge this pull request.

Assignees

@cav71 cav71

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@cav71