Closed mattt closed 3 years ago
@Lukas-Stuehrk It looks like I missed a few things in my review, so I'm including some cleanup commits here (ba00ca94d0f512a5d986fd69cc4aef5dd192e795, 7b83771d05477058afdcc7c5b6057da8b054d96f). Please take a look and let me know if we follow through on these changes.
No changelog entry required, as this isn't a user-facing change. Merging now.
Following up on #30, this PR adds equivalent tests for equality and hashing behavior for
Document
to bring it in line withNode
.