Skip to content

api/types/events: add "Type" type for event-type enum#42770

Merged
AkihiroSuda merged 1 commit intomoby:masterfrom
thaJeztah:eventtype_enums
Aug 27, 2021
Merged

api/types/events: add "Type" type for event-type enum#42770
AkihiroSuda merged 1 commit intomoby:masterfrom
thaJeztah:eventtype_enums

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

Currently just an alias for string, but we can change it to be an actual type.

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

Comment thread api/types/events/events.go Outdated
Comment on lines 15 to 18
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hm... I should've sorted this list while I was changing this.

Let me get that sorted (pun intended) before we merge this 😅

Currently just an alias for string, but we can change it to be  an
actual type.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah
Copy link
Copy Markdown
Member Author

Sorted!

Hope the Jenkins gods shine on me, and that CI will be green again (not hitting a flaky) 🤗

@thaJeztah
Copy link
Copy Markdown
Member Author

@AkihiroSuda ptal

@AkihiroSuda AkihiroSuda merged commit 0cd1bd4 into moby:master Aug 27, 2021
@thaJeztah thaJeztah deleted the eventtype_enums branch August 27, 2021 15:24
@thaJeztah thaJeztah added this to the 21.xx milestone Oct 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants