ChiselTest Compatibility Layer for Chisel 7#5221
ChiselTest Compatibility Layer for Chisel 7#5221EhsanKhodadad wants to merge 8 commits intochipsalliance:mainfrom
Conversation
|
Howdy @EhsanKhodadad, thank you for the contribution! Can you sign the CLA? |
Hi Jack, thank you for your comment. |
jackkoenig
left a comment
There was a problem hiding this comment.
Thank you again for doing this and for following up on the CLA. I have some comments.
I don't have a very good way to see how well this work in migrating from Chisel 6 with chiseltest to Chisel 7 with this. Do you have some example open-source code where we can see the code diff to perhaps use as a motivating example?
We have tested our layer in the Chisel-book repo, where there are several Chisel tests: |
move examples to test tree remove misleading DecoupledDriver/object exports avoid formal stubs
Contributor Checklist
docs/src?Type of Improvement
Desired Merge Strategy
Release Notes
Reviewer Checklist (only modified by reviewer)
3.6.x,5.x, or6.xdepending on impact, API modification or big change:7.0)?Enable auto-merge (squash)and clean up the commit message.Create a merge commit.