Skip to content

Commit

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

* 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]>
Signed-off-by: Miki <[email protected]>
  • Loading branch information
Wielmany authored and AMoo-Miki committed Jul 12, 2023
1 parent 89ba1e0 commit 3cb9ee7
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 16 deletions.
16 changes: 2 additions & 14 deletions .github/workflows/bundler.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
name: Bundler

# trigger on every commit push and PR for all branches
on:
# trigger on every commit push and PR for all branches
push:
branches: ['**']
paths-ignore:
Expand Down Expand Up @@ -31,21 +30,10 @@ jobs:
make cluster.clean cluster.opensearch.build cluster.opensearch.start
- name: Use Node.js 14.x
uses: actions/setup-node@v3
uses: actions/setup-node@v1
with:
node-version: 14.x

# NPM started understanding yarn.lock file starting from v7
- name: Update NPM
shell: bash
run: |
export NPM_VERSION=$(npm -v)
export IS_NPM_SUITABLE=$(node -p "parseInt(process.env.NPM_ROOT, 10) >= 7")
if [ "$IS_NPM_SUITABLE" == "false" ]; then
echo "NPM needs upgrading!"
npm i npm@7 -g
fi
- name: Install
run: |
npm install
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
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 3cb9ee7

Please sign in to comment.