Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Fixed #493 include_paths, when only certain keys are included #499

Merged
merged 3 commits into from
Nov 14, 2024

Conversation

jlaba
Copy link

@jlaba jlaba commented Oct 26, 2024

Fixes #493 by checking if the key or a complete higher level is included. If only sibling keys of this level are included, this key is skipped.

@seperman seperman changed the base branch from master to dev October 31, 2024 16:55
@seperman
Copy link
Owner

seperman commented Nov 4, 2024

@jlaba Awesome. I will review it shortly.

Copy link
Owner

@seperman seperman left a comment

Choose a reason for hiding this comment

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

LGTM!
Thank you for the PR.

@seperman seperman merged commit 32d60a9 into seperman:dev Nov 14, 2024
5 checks passed
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.

deepdiff with include-paths is changing results, if a not considered path changes
2 participants