-
Notifications
You must be signed in to change notification settings - Fork 8.2k
Fix #1986: Migrate Travis CI to GitHub Actions #1992
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
| architecture: x64 | ||
|
|
||
| - name: Build with Maven | ||
| run: mvn install -DskipTests=true -Dmaven.javadoc.skip=true -B -V -DminimumPriority=1 |
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.
Here we'll need to execute both build and test.
FYI (config in Travis CI): https://docs.travis-ci.com/user/languages/java/#maven-dependency-management
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, I will change it later.
sczyh30
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
|
Thanks for contributing! |
Add GitHub Actions CI workflow (alibaba#1992)
[ISSUE alibaba#1992] fix Broker2Client logs
Describe what this PR does / why we need it
Migrate Travis CI to GitHub Actions
Does this pull request fix one issue?
Fix #1986
Describe how you did it
migrate the old .travis.yml to .github/workflows/ci.yml
Describe how to verify it
check the Actions
Special notes for reviews