-
Notifications
You must be signed in to change notification settings - Fork 2.8k
fix hpc combine error (close #2946) #2947
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
Conversation
|
Deploy preview for hasura-docs ready! Built with commit d3d46f0 |
|
|
jberryman
left a comment
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.
Seems good to me except for adding that comment!
shahidhk
left a comment
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.
LGTM
|
Review app https://hge-ci-pull-2947.herokuapp.com is deleted |
* Fix hpc combine error * Do not perform ciignore * xfail test jsonb_has_all * Bring back ciignore * Refer jsonb_has_all xfaul to the corresponding issue in graphql-engine-internal
fix #2946
Description
Combining hpc reports fail with the following error
Affected components
Related Issues
Solution and Design
unset GHRTS during hpc combine, and restore after hpc combine
Steps to test and verify
Check the hpc combine step of circleci
Limitations, known bugs & workarounds