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

Bring kord enum API closer to bit flags API #861

Merged
merged 1 commit into from
Sep 3, 2023
Merged

Conversation

lukellmann
Copy link
Member

This was done by only printing the raw value for Unknown entries in toString() and replacing the public Unknown constructor with a from() function in the companion object.

This was done by only printing the raw value for Unknown entries in
toString() and replacing the public Unknown constructor with a from()
function in the companion object.
@lukellmann lukellmann requested a review from DRSchlaubi August 27, 2023 19:33
@lukellmann
Copy link
Member Author

The function is named fromShift in the generated bit flag classes, but i think fromValue / fromCode etc. is a bit redundant, the parameter is already named like this.

@lukellmann lukellmann merged commit ffa8390 into main Sep 3, 2023
6 checks passed
@lukellmann lukellmann deleted the refine-kord-enum-api branch September 3, 2023 15:30
lukellmann added a commit that referenced this pull request Sep 3, 2023
lukellmann added a commit that referenced this pull request Dec 26, 2023
The changes are the same as in #861, I simply forgot ArchiveDuration
there.
lukellmann added a commit that referenced this pull request Dec 26, 2023
The changes are the same as in #861, I simply forgot ArchiveDuration
there.
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.

2 participants