-
Notifications
You must be signed in to change notification settings - Fork 54
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
chore(tests): Simplify imports #2467
base: master
Are you sure you want to change the base?
Conversation
You can find the image built from this PR at
Built from 9f9519f |
1a4d344
to
814b9e7
Compare
814b9e7
to
3ded04b
Compare
64f276f
to
49256dc
Compare
80d0dd0
to
f4cd398
Compare
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.
my intuition is that this test was never imported anywhere and that is why we could not catch it while merging the rln-v2 changes m
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.
Agree, those were my findings as well
Description
Simplify test imports
Changes
test_all
filesall_test_waku
usingtest_all
files