-
Notifications
You must be signed in to change notification settings - Fork 6
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
Upgrade bulletin version to 0.24.4 #309
Conversation
…yodide-17 # Conflicts: # Dockerfile # Dockerfile.release # bulletin_board/server/db/seeds.rb # voting_schemes/electionguard/docker/pyodide-electionguard/Dockerfile # voting_schemes/electionguard/docker/pyodide-electionguard/packages/libgmp/Makefile # voting_schemes/electionguard/docker/pyodide-electionguard/packages/libgmp/checksums # voting_schemes/electionguard/python-to-js/build # voting_schemes/electionguard/python-wrapper/Pipfile.lock # voting_schemes/electionguard/python-wrapper/integration_results.jsonl # voting_schemes/electionguard/python-wrapper/src/bulletin_board/electionguard/trustee.py # voting_schemes/electionguard/python-wrapper/tests/test_integration.py
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #309 +/- ##
========================================
Coverage 69.57% 69.57%
========================================
Files 96 96
Lines 1630 1630
========================================
Hits 1134 1134
Misses 496 496 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't actually run the code changes directly, but as this is the version from docker hub and you have been using that for a couple weeks I think we can be sure that it's working 😄
Really outstanding documentation, thanks for the effort. I have some changes in format and we're good to go
voting_schemes/electionguard/python-wrapper/src/bulletin_board/electionguard/bulletin_board.py
Outdated
Show resolved
Hide resolved
Changes applied @andreslucena ! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏽
Depends on #308