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.

Sign up for GitHub

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

Jump to bottom

Make class Message unhashable #262

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Make class Message unhashable #262

wants to merge 3 commits into from

Commits on Mar 3, 2019

  1. Add __hash__ to class Message to make it hashable in Python 3 

    The class Message defines a __eq__ method, but, does not define a
__hash__ method. This makes it unhashable in Python 3. But, it is
hashable in Python 2.
    @ttanay
    ttanay committed Mar 3, 2019
    Configuration menu
    Copy the full SHA
    87f96a6 View commit details
    Browse the repository at this point in the history

Commits on Mar 11, 2019

  1. Make __hash__ implementation consistent with __eq__

    @ttanay
    ttanay committed Mar 11, 2019
    Configuration menu
    Copy the full SHA
    0ace821 View commit details
    Browse the repository at this point in the history

  2. Make __hash__ None for class Message

    @ttanay
    ttanay committed Mar 11, 2019
    Configuration menu
    Copy the full SHA
    ea930a3 View commit details
    Browse the repository at this point in the history