Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Release 0.1.3. Update dependencies to recent packages. #36

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nicolas-graves
Copy link

Hi !

I was working on the bootstrapping of rust packages on guix, and have found that this package makes the bootstrapping more difficult because of its old dependencies. I've updated the dependencies and made everything stick together so that test files would pass without being modified.

The structs in block.rs are strictly the same, expect for After and Before which became both vectors of statements instead of vectors of blocks. The logic stays the same with some light adaptations. In particular, there is no Root anymore, the speculate top-level is simply considered as a describe block.

Normally tests should pass properly, I'm also going to test this on dependent packages, if I don't come back with more info in the coming month, then it works.

@nicolas-graves nicolas-graves force-pushed the master branch 4 times, most recently from 74ad852 to be951a3 Compare May 10, 2023 19:32
@nicolas-graves
Copy link
Author

I had an error with the tests for diff, but it is fixed now. @utkarshkukreti I think it's good for a review.

@nicolas-graves nicolas-graves force-pushed the master branch 2 times, most recently from 7457160 to 0323780 Compare May 10, 2023 22:33
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant