-
Notifications
You must be signed in to change notification settings - Fork 178
HBASE-24452 Try github ci for hbase-connectors #69
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
HorizonNet
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.
I think maven.yml would be a better file name in case we want to add other Actions. Initial definition LGTM for a first try. Shouldn't this one gets already triggered on this PR?
I don't know whether this should trigger PR...... Let me rename it. |
HorizonNet
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.
Let's give it a try. I was under the impression that an Action will run right from the beginning, but that doesn't seem to be the case.
busbey
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.
I have noted my concern on the jira
|
🎊 +1 overall
This message was automatically generated. |
|
Do we need this change or can I close it? |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
No description provided.