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

Use object.__hash__ for Node.__hash__ #1522

Merged
merged 1 commit into from Nov 9, 2021
Merged

Commits on Nov 9, 2021

  1. Use object.__hash__ for Node.__hash__

    This fixes a regression in commit 6029341 that
    changed the `__hash__` implementation of Node from the default pointer
    hash, to a hash based on the node fields.
    
    Since these fields contains list objects, they are not hashable, making
    every call to `Node.__hash__` fail.
    
    This breaks some third-party usage such as in `django-compressor`
    (See: django-compressor/django-compressor#1060)
    
    This changed reverts the hash method back to using `object.__hash__` as
    the hash implementation.
    klette authored and davidism committed Nov 9, 2021
    Copy the full SHA
    9b96b48 View commit details
    Browse the repository at this point in the history