-
Notifications
You must be signed in to change notification settings - Fork 17
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
test downstream projects #57
Conversation
.github/workflows/ci.yml
Outdated
|
||
downstream_project: [meshmode, pytential, pytato] |
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.
meshmode and pytential aren't downstream of grudge?
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.
Ok, but which projects would you like to see tested here? I'm not sure that pytato is downstream of grudge either.
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.
Only mirgecom is downstream of grudge.
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.
Ok, so does it make sense to merge this PR, considering that illinois-ceesd/mirgecom#212 is still open?
Unsubscribing... @-mention or request review once it's ready for a look or needs attention. |
Current error is:
might be due to illinois-ceesd/mirgecom#212 ? @inducer |
Thx! |
Fixes #44.