chore: removed travis yml and added git action support#469
Conversation
Pull Request Test Coverage Report for Build 1979
💛 - Coveralls |
jaeopt
left a comment
There was a problem hiding this comment.
Looks good. A couple of suggestions.
.github/workflows/java.yml
Outdated
| echo "$GITHUB_CONTEXT" | ||
| home/runner/travisci-tools/trigger-script-with-status-update.sh | ||
| test: | ||
| runs-on: macos-latest |
There was a problem hiding this comment.
I see this is not fixed yet.
.github/workflows/java.yml
Outdated
| BUILD_NUMBER: ${{ github.run_id }} | ||
| GITHUB_CONTEXT: ${{ toJson(github) }} | ||
| EVENT_NAME: ${{ github.event_name }} | ||
| REPOSITORY: ${{ github.repository }} | ||
| SHA: ${{ github.sha }} | ||
| TRAVIS_PULL_REQUEST_SHA: ${{ github.event.pull_request.head.sha }} | ||
| TRAVIS_PULL_REQUEST: ${{ github.event.pull_request.number }} | ||
| EVENT_MESSAGE: ${{ github.event.message }} | ||
| HEAD_REF: ${{ github.head_ref }} | ||
| REF_NAME: ${{ github.ref_name }} | ||
| secrets: | ||
| CI_USER_TOKEN: ${{ secrets.CI_USER_TOKEN }} | ||
| TRAVIS_COM_TOKEN: ${{ secrets.TRAVIS_COM_TOKEN }} |
There was a problem hiding this comment.
I see factoring out to "integration_test.yml" looks good, but not much gain (more overhead than sharing). I'll leave it up to you. I guess you already tried, but wondering if we cannot use ${{ github }} directly in "integration_test.yml" instead of passing as input parameters.
There was a problem hiding this comment.
Yeah, i actually asked for it that cleans-up our main.yaml file.
Regarding your suggestion we tried but github is very specific to passing secrets info to actions, that's why this was not done.
.github/workflows/java.yml
Outdated
| echo "$GITHUB_CONTEXT" | ||
| home/runner/travisci-tools/trigger-script-with-status-update.sh | ||
| test: | ||
| runs-on: macos-latest |
There was a problem hiding this comment.
I see this is not fixed yet.
|
@mnoman09 I tried to release android-sdk with actions flow first time, and found a couple of errors. We can do the same fix for java-sdk. |
|
@mnoman09 Also found that "publish" task is launched even before unit tests completed. Either we can make it wait until all tests completed or we can skip them for publish (since we already done the same tests before publish). |
| @@ -0,0 +1,16 @@ | |||
| name: Source clear | |||
|
|
|||
| on: | |||
There was a problem hiding this comment.
can we add master branch here.
Summary
NOTE:
Passed build link:
Test plan
All tests should pass