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
11 changes: 10 additions & 1 deletion .github/workflows/bundler.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,15 @@
name: Bundler

on: [push, pull_request]
# trigger on every commit push and PR for all branches except pushes for backport branches
on:
push:
branches: ['**', '!backport/**']
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 except pushes for backport branches
on:
push:
branches: ['**', '!backport/**']
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 except pushes for backport branches
on:
push:
branches: ['**', '!backport/**']
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 except pushes for backport branches
on:
push:
branches: ['**', '!backport/**']
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 except pushes for backport branches
on:
push:
branches: ['**', '!backport/**']
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
3 changes: 2 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
### Changed
- 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))
- 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
### Removed
- Remove waitCluster in Integration Tests ([#422](https://github.com/opensearch-project/opensearch-js/issues/422))
Expand Down Expand Up @@ -86,4 +87,4 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)

[2.1]: https://github.com/opensearch-project/opensearch-js/releases/tag/2.1.0
[2.2]: https://github.com/opensearch-project/opensearch-js/releases/tag/2.2.0
[Unreleased]: https://github.com/opensearch-project/opensearch-js/compare/2.2...HEAD
[Unreleased]: https://github.com/opensearch-project/opensearch-js/compare/2.2...HEAD