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

fix: lock ursa-bbs-signatures to 1.0.1 #1704

Closed
wants to merge 2 commits into from

Conversation

TimoGlastra
Copy link
Contributor

Pin ursa-bbs-signatures to 1.0.1 as 1.0.2 has broken the tests as also described here.

We've just discovered a 1.0.2. release was made yesterday by mistake and we're currently working on un-releasing the package or making a 1.0.3 release that fixes this issue.

Signed-off-by: Timo Glastra <timo@animo.id>
@swcurran
Copy link
Contributor

@shaangill025 -- for you to manage how this gets into ACA-Py with minimal impact on the rest of the PRs.

@codecov-commenter
Copy link

Codecov Report

Merging #1704 (b507f3b) into main (bdba6b5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1704   +/-   ##
=======================================
  Coverage   95.29%   95.29%           
=======================================
  Files         528      528           
  Lines       32906    32906           
=======================================
  Hits        31358    31358           
  Misses       1548     1548           

@shaangill025
Copy link
Contributor

shaangill025 commented Mar 30, 2022

@TimoGlastra This change was merged into main from PR #1690. You can close this one and let us know when we can update to a newer release.

@karimStekelenburg
Copy link
Contributor

I have removed the 1.0.2 version from PyPi. This should hopefully solve the issue.

@TimoGlastra TimoGlastra closed this Apr 3, 2022
@TimoGlastra TimoGlastra deleted the fix/use-ursa-bbs-signatures-101 branch April 3, 2022 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants