-
Notifications
You must be signed in to change notification settings - Fork 8
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
non-product upwinding #1367
base: main
Are you sure you want to change the base?
non-product upwinding #1367
Conversation
bors try |
tryBuild failed: |
bors try |
tryBuild failed: |
bors try |
tryBuild failed: |
bors try |
tryBuild failed: |
bors try |
tryBuild failed: |
bors try |
tryBuild failed: |
bors try |
tryBuild succeeded! The publicly hosted instance of bors-ng is deprecated and will go away soon. If you want to self-host your own instance, instructions are here. If you want to switch to GitHub's built-in merge queue, visit their help page. |
bors try |
bors try- |
ba84079
to
efc543c
Compare
bors try |
tryBuild succeeded! The publicly hosted instance of bors-ng is deprecated and will go away soon. If you want to self-host your own instance, instructions are here. If you want to switch to GitHub's built-in merge queue, visit their help page. |
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.
Are there tests added for the new operator(s)? It's a bit hard to tell with the name changes squashed together. If not, maybe we can add them / convergence tests? Aside from that LGTM
Good point. I was relying on the implicit testing, but we should add some more |
81187ca
to
237fce1
Compare
bors try |
tryBuild failed: |
237fce1
to
224eb62
Compare
bors try |
tryBuild failed: |
224eb62
to
6a00f21
Compare
6a00f21
to
701f7ee
Compare
Fixes #1125.