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

IP addon finder mac address format tweak #4481

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

Conversation

andrewfg
Copy link
Contributor

@andrewfg andrewfg commented Dec 8, 2024

Tweaks to #4477

Signed-off-by: Andrew Fiddian-Green [email protected]

Signed-off-by: Andrew Fiddian-Green <[email protected]>
@andrewfg andrewfg requested a review from a team as a code owner December 8, 2024 13:30
@andrewfg
Copy link
Contributor Author

andrewfg commented Dec 8, 2024

@holgerfriedrich ping!!

@holgerfriedrich
Copy link
Member

@andrewfg Sorry, I did not manage to review this before Kai pressed the button to build M5. Shouldn't be a problem to delay this, as the merged implementation works for the common delimiters.

@holgerfriedrich holgerfriedrich added the enhancement An enhancement or new feature of the Core label Dec 9, 2024
@andrewfg
Copy link
Contributor Author

andrewfg commented Dec 9, 2024

Yes. It really only affects developers who might write faulty code. And probably such developers will be using snapshots anyway.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature of the Core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants