Skip to content
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

[CCI] Don't run tests on PRs with doc changes only #441

Merged
merged 9 commits into from
Mar 22, 2023
Merged
10 changes: 9 additions & 1 deletion .github/workflows/bundler.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,14 @@
name: Bundler

on: [push, pull_request]
# trigger on every commit push and PR for all branches
push:
branches: ['**']
paths-ignore:
- '**/*.md'
pull_request:
branches: ['**']
paths-ignore:
- '**/*.md'

jobs:
bundler-support:
Expand Down
11 changes: 10 additions & 1 deletion .github/workflows/compatibility.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,15 @@
name: Integration for Compatibility

on: [push, pull_request]
# trigger on every commit push and PR for all branches
on:
push:
branches: ['**']
paths-ignore:
- '**/*.md'
pull_request:
branches: ['**']
paths-ignore:
- '**/*.md'

jobs:
integration-test-compatibility:
Expand Down
12 changes: 10 additions & 2 deletions .github/workflows/coverage.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,15 @@
name: Coverage

on: [push, pull_request]

# trigger on every commit push and PR for all branches
on:
push:
branches: ['**']
paths-ignore:
- '**/*.md'
pull_request:
branches: ['**']
paths-ignore:
- '**/*.md'
jobs:
code-coverage:
name: Code coverage
Expand Down
4 changes: 4 additions & 0 deletions .github/workflows/integration-unreleased.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,13 @@ on:
push:
branches:
- 'main'
paths-ignore:
- '**/*.md'
pull_request:
branches:
- 'main'
paths-ignore:
- '**/*.md'

jobs:
test:
Expand Down
12 changes: 10 additions & 2 deletions .github/workflows/integration.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,14 @@
name: Integration

on: [push, pull_request]
# trigger on every commit push and PR for all branches
on:
push:
branches: ['**']
paths-ignore:
- '**/*.md'
pull_request:
branches: ['**']
paths-ignore:
- '**/*.md'

jobs:
helpers-integration-test:
Expand Down
15 changes: 12 additions & 3 deletions .github/workflows/nodejs.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,15 @@
name: Node CI

on: [push, pull_request]
# trigger on every commit push and PR for all branches
on:
push:
branches: ['**']
paths-ignore:
- '**/*.md'
pull_request:
branches: ['**']
paths-ignore:
- '**/*.md'

jobs:
test:
Expand All @@ -27,8 +36,8 @@ jobs:
- name: Setup Yarn
run: |
npm uninstall -g yarn
npm i -g [email protected]
npm i -g [email protected]

- name: Lint
if: matrix.node-version == '16.x'
run: |
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
- Remove test artifacts from gh_pages workflow ([#335](https://github.com/opensearch-project/opensearch-js/issues/335))
- Make fields in `BulkOperation` optional to match OpenSearch Bulk API requirements ([#378](https://github.com/opensearch-project/opensearch-js/pull/378))
- Remove guidance on using npm and switch completely to yarn in developer_guide ([#439](https://github.com/opensearch-project/opensearch-js/issues/435))
- Change coverage, compatability, integration, integration with unreleased Open Search, node ci, bundler tests not to run on documentation change ([441](https://github.com/opensearch-project/opensearch-js/pull/441))

### Deprecated

Expand Down