GitHub Actions Cleanup (#1467)
Summary: Pull Request resolved: https://github.com/facebook/yoga/pull/1467 1. Avoid reusing workflow names between validation and publish jobs, since they show up with the same UI when wanting to adhoc trigger a publish. 2. Merge SwiftPM checks into Validate Apple job 3. Merge remnants of website-next workflow into website workflow (and remove a redundant explicit build step already removed in one workflow) 4. Run more validation on push to release branch 5. Do not run website validation off of main, since it is only ever deployed from main branch Reviewed By: cipolleschi Differential Revision: D51236885 fbshipit-source-id: dd8165aa9871f173d8914f345569c6cde1edda72
This commit is contained in:
committed by
Facebook GitHub Bot
parent
6f41a27e31
commit
8c1a672f92
2
.github/workflows/publish-website.yml
vendored
2
.github/workflows/publish-website.yml
vendored
@@ -1,4 +1,4 @@
|
||||
name: Website
|
||||
name: Publish Website
|
||||
|
||||
on:
|
||||
push:
|
||||
|
3
.github/workflows/validate-android.yml
vendored
3
.github/workflows/validate-android.yml
vendored
@@ -1,10 +1,11 @@
|
||||
name: Android
|
||||
name: Validate Android
|
||||
|
||||
on:
|
||||
pull_request:
|
||||
push:
|
||||
branches:
|
||||
- main
|
||||
- 'release-*'
|
||||
workflow_dispatch:
|
||||
|
||||
jobs:
|
||||
|
18
.github/workflows/validate-apple.yml
vendored
18
.github/workflows/validate-apple.yml
vendored
@@ -1,15 +1,16 @@
|
||||
name: Apple
|
||||
name: Validate Apple
|
||||
|
||||
on:
|
||||
pull_request:
|
||||
push:
|
||||
branches:
|
||||
- main
|
||||
- 'release-*'
|
||||
workflow_dispatch:
|
||||
|
||||
jobs:
|
||||
lint-pods:
|
||||
name: Lint pod
|
||||
name: Build [CocoaPods]
|
||||
runs-on: macos-latest
|
||||
|
||||
steps:
|
||||
@@ -20,3 +21,16 @@ jobs:
|
||||
|
||||
- name: pod lib lint
|
||||
run: pod lib lint --verbose --include-podspecs=**/*.podspec
|
||||
|
||||
test:
|
||||
name: Build [SwiftPM]
|
||||
runs-on: macos-latest
|
||||
|
||||
steps:
|
||||
- uses: actions/checkout@v3
|
||||
|
||||
- name: Build Debug
|
||||
run: swift build -c debug
|
||||
|
||||
- name: Build Release
|
||||
run: swift build -c release
|
||||
|
3
.github/workflows/validate-cpp.yml
vendored
3
.github/workflows/validate-cpp.yml
vendored
@@ -1,10 +1,11 @@
|
||||
name: C++
|
||||
name: Validate C++
|
||||
|
||||
on:
|
||||
pull_request:
|
||||
push:
|
||||
branches:
|
||||
- main
|
||||
- 'release-*'
|
||||
workflow_dispatch:
|
||||
|
||||
env:
|
||||
|
3
.github/workflows/validate-js.yml
vendored
3
.github/workflows/validate-js.yml
vendored
@@ -1,10 +1,11 @@
|
||||
name: JavaScript
|
||||
name: Validate JavaScript
|
||||
|
||||
on:
|
||||
pull_request:
|
||||
push:
|
||||
branches:
|
||||
- main
|
||||
- 'release-*'
|
||||
workflow_dispatch:
|
||||
|
||||
env:
|
||||
|
21
.github/workflows/validate-python.yml
vendored
21
.github/workflows/validate-python.yml
vendored
@@ -1,21 +0,0 @@
|
||||
name: Python
|
||||
|
||||
on:
|
||||
pull_request:
|
||||
push:
|
||||
branches:
|
||||
- main
|
||||
workflow_dispatch:
|
||||
|
||||
jobs:
|
||||
format:
|
||||
name: Format
|
||||
runs-on: ubuntu-latest
|
||||
|
||||
steps:
|
||||
- uses: actions/checkout@v3
|
||||
|
||||
- name: black --check
|
||||
uses: ./.github/actions/black
|
||||
with:
|
||||
directory: ${{ github.workspace }}
|
22
.github/workflows/validate-swiftpm.yml
vendored
22
.github/workflows/validate-swiftpm.yml
vendored
@@ -1,22 +0,0 @@
|
||||
name: Swift
|
||||
|
||||
on:
|
||||
pull_request:
|
||||
push:
|
||||
branches:
|
||||
- main
|
||||
workflow_dispatch:
|
||||
|
||||
jobs:
|
||||
test:
|
||||
name: Build
|
||||
runs-on: macos-latest
|
||||
|
||||
steps:
|
||||
- uses: actions/checkout@v3
|
||||
|
||||
- name: Build Debug
|
||||
run: swift build -c debug
|
||||
|
||||
- name: Build Release
|
||||
run: swift build -c release
|
37
.github/workflows/validate-website-next.yml
vendored
37
.github/workflows/validate-website-next.yml
vendored
@@ -1,37 +0,0 @@
|
||||
name: Website (Next)
|
||||
|
||||
on:
|
||||
pull_request:
|
||||
push:
|
||||
branches:
|
||||
- main
|
||||
workflow_dispatch:
|
||||
|
||||
build:
|
||||
name: Build
|
||||
runs-on: ubuntu-latest
|
||||
|
||||
steps:
|
||||
- uses: actions/checkout@v3
|
||||
|
||||
- name: Setup
|
||||
uses: ./.github/actions/setup-js
|
||||
|
||||
- name: Build Website
|
||||
run: yarn build
|
||||
working-directory: website-next
|
||||
|
||||
typecheck:
|
||||
name: Typecheck
|
||||
runs-on: ubuntu-latest
|
||||
|
||||
steps:
|
||||
- uses: actions/checkout@v3
|
||||
|
||||
- name: yarn install
|
||||
run: yarn install --frozen-lockfile
|
||||
working-directory: javascript
|
||||
|
||||
- name: yarn tsc
|
||||
run: yarn tsc
|
||||
working-directory: website-next
|
22
.github/workflows/validate-website.yml
vendored
22
.github/workflows/validate-website.yml
vendored
@@ -1,7 +1,9 @@
|
||||
name: Website
|
||||
name: Validate Website
|
||||
|
||||
on:
|
||||
pull_request:
|
||||
branches:
|
||||
- main
|
||||
push:
|
||||
branches:
|
||||
- main
|
||||
@@ -32,10 +34,20 @@ jobs:
|
||||
- name: Setup
|
||||
uses: ./.github/actions/setup-js
|
||||
|
||||
- name: Build Yoga
|
||||
run: yarn build
|
||||
working-directory: javascript
|
||||
|
||||
- name: Build Website
|
||||
run: yarn build
|
||||
working-directory: website-next
|
||||
|
||||
typecheck:
|
||||
name: Typecheck [Docusaurus]
|
||||
runs-on: ubuntu-latest
|
||||
|
||||
steps:
|
||||
- uses: actions/checkout@v3
|
||||
|
||||
- name: Setup
|
||||
uses: ./.github/actions/setup-js
|
||||
|
||||
- name: yarn tsc
|
||||
run: yarn tsc
|
||||
working-directory: website-next
|
||||
|
Reference in New Issue
Block a user