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
PHOENIX-5234 Create patch scripts for phoenix-connectors and phoenix-queryserver projects #6
Conversation
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.
Thanks for the contribution @nsgupta1! This is very useful. Please take a look at the review comments. Can you please post a screenshot of what a possible green/red build would look like?
Also, if mvn clean install
fails for a PR, does this prevent the PR from being merged, or is it more like a warning?
If checks from PR are failing we get warnings like this example, It doesn't block from merging the PR. |
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.
A small nit comment, otherwise lgtm.
@nsgupta1 The URL (https://github.com/nsgupta1/Test_Action_tab/pull/2) is not accessible. Did you want to put up a public gist instead? |
Overall +1. Please squash all commits before we can merge. Thanks @nsgupta1 |
3b7b295
to
517db23
Compare
…-queryserver projects
This github workflow adds a check to run phoenix-connectors integration test based on every PR and push to master branch. Authors/reviewers can view the result of it in "check" tab of PR. @vincentpoon @ChinmaySKulkarni @twdsilva can you please look into this ?