Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Fix/asdict recurrence #202

Merged
merged 7 commits into from
Mar 28, 2023
Merged

Fix/asdict recurrence #202

merged 7 commits into from
Mar 28, 2023

Conversation

kagrski
Copy link
Collaborator

@kagrski kagrski commented Mar 28, 2023

Closes #199

cicharka
cicharka previously approved these changes Mar 28, 2023
View reviewed changes
Copy link
Collaborator

@cicharka cicharka left a comment

Choose a reason for hiding this comment

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

Works well

sbasan
sbasan previously approved these changes Mar 28, 2023
View reviewed changes
Copy link
Collaborator

@sbasan sbasan left a comment

Choose a reason for hiding this comment

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

lgtm

@kagrski kagrski enabled auto-merge (squash) March 28, 2023 15:41
@kagrski kagrski merged commit 5a87f4b into main Mar 28, 2023
@kagrski kagrski deleted the fix/asdict_recurrence branch March 28, 2023 16:22
@JimOverholt JimOverholt mentioned this pull request Apr 25, 2024
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@JimOverholt JimOverholt JimOverholt approved these changes

@cicharka cicharka cicharka approved these changes

@sbasan sbasan sbasan left review comments

@igosoft igosoft Awaiting requested review from igosoft igosoft is a code owner

@bboot2 bboot2 Awaiting requested review from bboot2 bboot2 is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

asdict() method is not recursive
4 participants
@kagrski @cicharka @JimOverholt @sbasan