amery
/
jpictl
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity
Labels Milestones

rings: introduce RingID and DecodeAddress() #50

Merged
amery merged 3 commits from pr-amery-rings into main 7 months ago
Conversation 1 Commits 3 Files Changed 3
amery commented 7 months ago
Owner
There is no content yet.
amery added 5 commits 7 months ago
83921c1e13 build-sys: use revive 1.3.7 instead of master
ff8f2c6ea1 rings: add String() to RegionID, ZoneID, NodeID
6d89e0ea3c rings: fix NodeZeroMax/NodeMax to not accept .255
169a1e9602 rings: introduce RingID and its values
amery requested review from karasz 7 months ago
amery added 1 commit 7 months ago
bcb20ab1e6 rings: introduce ring-specific decoders
karasz approved these changes 7 months ago
karasz left a comment
Owner

LGTM

LGTM
🎉 1
amery merged commit 879ee69f07 into main 7 months ago
amery deleted branch pr-amery-rings 7 months ago
amery referenced this issue from a commit 7 months ago
Merge pull request #50
amery commented 7 months ago
Poster
Owner

v0.8.3 tagged

`v0.8.3` tagged

Reviewers

karasz approved these changes 7 months ago
The pull request has been merged as 879ee69f07.
Sign in to join this conversation.
Reviewers
Request review
No reviewers
karasz
Labels
Apply labels
Clear labels
No items
No Label
Milestone
Set milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Assign users
Clear assignees
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch 'pr-amery-rings'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes