Skip to content

name + bitrange fix #172

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

name + bitrange fix #172

wants to merge 2 commits into from

Conversation

Emilgardis
Copy link
Member

  • check for wrong range size
  • add function to get name of cluster

@Emilgardis Emilgardis requested a review from a team as a code owner November 9, 2021 21:46
@Emilgardis Emilgardis mentioned this pull request Nov 9, 2021
Draft
@Emilgardis
Copy link
Member Author

the name() fn can probably be discarded.

@burrbull
Copy link
Member

the name() fn can probably be discarded.

as we have Deref, &self.name should already work

@Emilgardis Emilgardis closed this Nov 10, 2021
@Emilgardis Emilgardis deleted the name branch November 10, 2021 18:28
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Emilgardis @burrbull