Skip to content

[dpd-api] migrate to RFD 619 pattern#259

Draft
sunshowers wants to merge 4 commits intomainfrom
sunshowers/spr/dpd-api-migrate-to-rfd-619-pattern
Draft

[dpd-api] migrate to RFD 619 pattern#259
sunshowers wants to merge 4 commits intomainfrom
sunshowers/spr/dpd-api-migrate-to-rfd-619-pattern

Conversation

@sunshowers
Copy link
Copy Markdown
Contributor

No description provided.

Created using spr 1.3.6-beta.1
Created using spr 1.3.6-beta.1
Created using spr 1.3.6-beta.1
Created using spr 1.3.6-beta.1
Comment thread dpd-api/src/lib.rs
Comment on lines +524 to +529
//
// TODO: `FreeChannels` is unchanged across versions, so this split may be
// unnecessary — the rationale for gating the endpoint at
// `VERSION_MCAST_STRICT_UNDERLAY` is not obvious from the code alone.
// Revisit in a follow-up to either document the reason or collapse back to
// a single `channels_list` endpoint.
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Wanted to ask about this -- is this correct?

Comment thread dpd-api/src/lib.rs
Comment on lines -1562 to +1512
versions = VERSION_ATTACHED_SUBNETS..,
versions = VERSION_ATTACHED_SUBNETS..,
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

There were a few tabs which I switched over to spaces.

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.

1 participant