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

Agent event follow-up fixes for #442 #454

Merged
merged 2 commits into from
Dec 10, 2020

Conversation

tklauser
Copy link
Member

Address two review comments for #442 which I forgot to push before merging:

#442 (comment)
#442 (comment)

@tklauser tklauser added 🧹 kind/cleanup This includes no functional changes. release-note/misc This PR makes changes that have no direct user impact. labels Dec 10, 2020
@tklauser tklauser requested a review from rolinh December 10, 2020 09:58
Address #442 (comment)

This will make sure that IPv6 addresses are formatted properly.

Fixes: 94b38af ("printer: add basic support for AgentEvent")
Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
Addresses #442 (comment)

Fixes: 94b38af ("printer: add basic support for AgentEvent")
Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
@tklauser tklauser force-pushed the pr/tklauser/agent-event-follow-up branch from 695dee9 to dec6459 Compare December 10, 2020 10:00
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Dec 10, 2020
@tklauser tklauser merged commit e65368f into master Dec 10, 2020
@tklauser tklauser deleted the pr/tklauser/agent-event-follow-up branch December 10, 2020 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧹 kind/cleanup This includes no functional changes. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants