-
Notifications
You must be signed in to change notification settings - Fork 592
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
refactor(ci): modularize the federated integration test make command #14170
refactor(ci): modularize the federated integration test make command #14170
Conversation
Thanks for opening a PR! 💯
Howto
More infoPlease take a moment to read through the Magma project's
If this is your first Magma PR, also consider reading
|
e0e3615
to
39cfa41
Compare
15f16f1
to
f695c0b
Compare
f695c0b
to
1650e04
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.
Changes LGTM, although I don't see (yet?) the real benefit of having a separate handle for the eval export
expression only. But this may be up to personal preference.
1650e04
to
8e12cce
Compare
Signed-off-by: Fritz Lehnert <13189449+Neudrino@users.noreply.github.com>
8e12cce
to
d297b16
Compare
|
Summary
Test Plan
Additional Information