Skip to content
This repository was archived by the owner on Jul 3, 2023. It is now read-only.

Conversation

@blp
Copy link
Contributor

@blp blp commented Feb 6, 2023

This is a resubmission of PR #286 with the additional fix commits from that PR already applied. It is being resubmitted because currently the CI/CD pipeline only works with branches in the main DBSP repo, whereas PR #286 came from my fork.

Signed-off-by: Ben Pfaff [email protected]

blp and others added 3 commits February 6, 2023 17:16
Add derive's for `Encode` and `Decode` in the `orgchart` example, so
that it compiles with persistence enabled.
I didn't understand that the difference between 'join' and 'join_index'
was that the latter indexes the output stream.  The example did not use
the index, so we can omit it.  This commit does so.

Signed-off-by: Ben Pfaff <[email protected]>
@ryzhyk ryzhyk merged commit 2b7b4d7 into main Feb 7, 2023
@ryzhyk ryzhyk deleted the orgchart branch February 7, 2023 01:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants