Yicong-Huang commented on code in PR #4278:
URL: https://github.com/apache/texera/pull/4278#discussion_r2914796970
##########
.github/workflows/github-action-build.yml:
##########
@@ -26,14 +26,39 @@ on:
- 'ci-enable/**'
- 'main'
pull_request:
+ types: [opened, synchronize, reopened]
workflow_dispatch:
concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
cancel-in-progress: ${{ github.ref != 'refs/heads/main' }}
jobs:
+ check-permissions:
+ runs-on: ubuntu-latest
+ steps:
+ - name: Check if PR is from a fork
+ id: check_fork
+ run: |
+ if [ "${{ github.event.pull_request.head.repo.full_name }}" != "${{
github.repository }}" ]; then
+ echo "is_fork=true" >> $GITHUB_OUTPUT
Review Comment:
I think for the long run, even committers should use fork to manage, instead
of pushing directly to the main repo. Currently we have too many personal used
branches on the main repo and we would want to clean them out.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
