Skip to content

Commit

Permalink
[CCI] Don't run tests on PRs with doc changes only (#441)
Browse files Browse the repository at this point in the history
* don't run coverage tests on doc changes

Signed-off-by: Didar Tursunov <[email protected]>

* don't run compatability tests on doc changes

Signed-off-by: Didar Tursunov <[email protected]>

* don't run integrations with unreleased OS test on doc change

Signed-off-by: Didar Tursunov <[email protected]>

* don't run integration checks on doc change

Signed-off-by: Didar Tursunov <[email protected]>

* don't run node ci tests on doc changes

Signed-off-by: Didar Tursunov <[email protected]>

* don't run bundler tests on doc changes

Signed-off-by: Didar Tursunov <[email protected]>

* Update changelog

Signed-off-by: Didar Tursunov <[email protected]>

* run tests on backport pushes

Signed-off-by: Didar Tursunov <[email protected]>

---------

Signed-off-by: Didar Tursunov <[email protected]>
  • Loading branch information
Wielmany authored Mar 22, 2023
1 parent 4a7fbeb commit ca93b2a
Show file tree
Hide file tree
Showing 7 changed files with 56 additions and 9 deletions.
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

0 comments on commit ca93b2a

Please sign in to comment.