Skip to content

Conversation

@tseaver
Copy link
Contributor

@tseaver tseaver commented Sep 29, 2021

Closes #251.

@tseaver tseaver requested a review from a team as a code owner September 29, 2021 17:02
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Sep 29, 2021
@codecov
Copy link

codecov bot commented Sep 29, 2021

Codecov Report

Merging #252 (44cde4a) into master (6a43682) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #252   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           20        20           
  Lines          862       922   +60     
  Branches       149       169   +20     
=========================================
+ Hits           862       922   +60     
Impacted Files Coverage Δ
proto/modules.py 100.00% <ø> (ø)
proto/enums.py 100.00% <100.00%> (ø)
proto/marshal/collections/repeated.py 100.00% <100.00%> (ø)
proto/marshal/marshal.py 100.00% <100.00%> (ø)
proto/message.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c65faf6...44cde4a. Read the comment docs.

@tseaver tseaver merged commit 232341b into master Sep 29, 2021
@tseaver tseaver deleted the 251-fix-enum-hash branch September 29, 2021 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enums became unhashable

2 participants