-
-
Notifications
You must be signed in to change notification settings - Fork 36
[Collections\dictionary] Migrate tests to unitt
#1854
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
base: master
Are you sure you want to change the base?
[Collections\dictionary] Migrate tests to unitt
#1854
Conversation
Ready to merge! 🚀 |
@drkameleon if you need me to push some random commit to rerun the CI, let me know. 😉 |
I had totally missed this one. And I have absolutely zero idea why the CI actions are stuck... Can you merge from the master and just commit/push? |
@RickBarretto Ping! lol 😄 |
@drkameleon Do you want me to push a random commit? I've finished this PR. And also, I'm using the older version of |
Yes, I think this could work. 😉 I mean, it's literally stuck like that lol |
@drkameleon alright, I'll push ASAP when I touch my computer. |
merely to trigger the CI action
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.
@RickBarretto In the end, I added a trivial commit myself to get it... rolling, but it appears it stumbles upon an error in the unit-test itself 🤔
Can you have a look into it so that we can merge it? 😉
Description
Migrate print based tests to
unitt
, decoupling dictionary behavior fromprint
function.Type of change