Skip to content

Commit

Permalink
Fix missing runner in Linux selective build and custom ops jobs (#313)
Browse files Browse the repository at this point in the history
Summary:
I mistakenly replace this in #277.  The runner is not set here, and causes syntax error, for example https://github.com/pytorch/executorch/actions/runs/6167285165.  The workflow still works because the default `linux.2xlarge` is used (same runner type)

### Testing

The annotation error is gone now https://github.com/pytorch/executorch/actions/runs/6175961826

Pull Request resolved: #313

Test Plan: https://github.com/pytorch/executorch/actions/runs/6175961826

Reviewed By: mergennachin

Differential Revision: D49236002

Pulled By: huydhn

fbshipit-source-id: bd5282f582d23427fd1d7622ed3b2ee2e6ad93cb
  • Loading branch information
huydhn authored and facebook-github-bot committed Sep 13, 2023
1 parent 63bb3b5 commit 1f558c7
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/pull.yml
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ jobs:
- build-tool: cmake
fail-fast: false
with:
runner: ${{ matrix.runner }}
runner: linux.2xlarge
docker-image: executorch-ubuntu-22.04-clang12
submodules: 'true'
ref: ${{ github.event_name == 'pull_request' && github.event.pull_request.head.sha || github.sha }}
Expand Down Expand Up @@ -148,7 +148,7 @@ jobs:
- build-tool: cmake
fail-fast: false
with:
runner: ${{ matrix.runner }}
runner: linux.2xlarge
docker-image: executorch-ubuntu-22.04-clang12
submodules: 'true'
ref: ${{ github.event_name == 'pull_request' && github.event.pull_request.head.sha || github.sha }}
Expand Down

0 comments on commit 1f558c7

Please sign in to comment.