Skip to content

fix port range to include end - #518

Merged
bee-san merged 2 commits into
bee-san:masterfrom
hpmaxi:bug/close-end-port-range
Apr 7, 2024
Merged

fix port range to include end#518
bee-san merged 2 commits into
bee-san:masterfrom
hpmaxi:bug/close-end-port-range

Conversation

@hpmaxi

@hpmaxi hpmaxi commented May 25, 2023

Copy link
Copy Markdown
Contributor

Closes #515
Use close range for test and range generation.

@bee-san

bee-san commented Apr 6, 2024

Copy link
Copy Markdown
Owner

Hi! Mind pushing an empty commit? CI has failed to pick this up and I think it'll only pick it up on a new commit? 馃

Signed-off-by: Maximiliano Molina <hpmaxi@gmail.com>
@hpmaxi

hpmaxi commented Apr 6, 2024

Copy link
Copy Markdown
Contributor Author

Hi! Mind pushing an empty commit? CI has failed to pick this up and I think it'll only pick it up on a new commit? 馃

Done! 1 workflow awaiting approval

@bee-san

bee-san commented Apr 7, 2024

Copy link
Copy Markdown
Owner

Hi! Mind pushing an empty commit? CI has failed to pick this up and I think it'll only pick it up on a new commit? 馃

Done! 1 workflow awaiting approval

Running!

@bee-san
bee-san merged commit 598b5df into bee-san:master Apr 7, 2024
@bee-san

bee-san commented Apr 7, 2024

Copy link
Copy Markdown
Owner

thank you!!

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.

--range <start-end>, only contain start, But not end port

2 participants