Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: opensearch-project/OpenSearch-Dashboards
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: main
Choose a base ref
...
head repository: opensearch-project/OpenSearch-Dashboards
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: backport/backport-8144-to-2.17
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
Loading
Showing 4,850 changed files with 422,376 additions and 149,939 deletions.
The diff you're trying to view is too large. We only load the first 3000 changed files.
15 changes: 10 additions & 5 deletions .browserslistrc
Original file line number Diff line number Diff line change
@@ -1,9 +1,14 @@
[production]
last 2 Firefox versions
last 2 Chrome versions
last 2 Safari versions
> 0.25%
not ie 11
Firefox > 0 and last 2 years and > 0.01%
Chrome > 0 and last 2 years and > 0.01%
Safari > 0 and last 2 years and > 0.01%
Edge > 0 and last 1 years and > 0.01%
Opera > 0 and last 2 years and > 0.01%
> 0.2%
not op_mini all
not and_uc < 100
not android < 100
not dead

[dev]
last 1 chrome versions
8 changes: 6 additions & 2 deletions .eslintignore
Original file line number Diff line number Diff line change
@@ -3,6 +3,7 @@
/.chromium
/build
/built_assets
/bwc_tmp
/config/apm.dev.js
/data
/html_docs
@@ -17,8 +18,7 @@ target
# plugin overrides
/src/core/lib/osd_internal_native_observable
/src/legacy/plugin_discovery/plugin_pack/__tests__/fixtures/plugins/broken
/src/plugins/data/common/opensearch_query/kuery/ast/_generated_/**
/src/plugins/vis_type_timeline/public/_generated_/**
/src/plugins/**/_generated_/**

# package overrides
/packages/opensearch-eslint-config-opensearch-dashboards
@@ -30,3 +30,7 @@ target
/packages/osd-test/src/functional_test_runner/lib/config/__tests__/fixtures/
/packages/osd-ui-framework/dist
/packages/osd-ui-shared-deps/flot_charts

# antlr overrides
/src/plugins/data/public/antlr/opensearch_sql/.generated/*
/src/plugins/data/public/antlr/opensearch_sql/grammar/**/*
49 changes: 27 additions & 22 deletions .eslintrc.js
Original file line number Diff line number Diff line change
@@ -108,8 +108,8 @@ const SAFER_LODASH_SET_DEFINITELYTYPED_HEADER = `
`;

const allMochaRulesOff = {};
Object.keys(require('eslint-plugin-mocha').rules).forEach((k) => {
allMochaRulesOff['mocha/' + k] = 'off';
Object.keys(require('eslint-plugin-mocha-next').rules).forEach((k) => {
allMochaRulesOff['mocha-next/' + k] = 'off';
});

module.exports = {
@@ -272,26 +272,22 @@ module.exports = {
basePath: __dirname,
zones: [
{
target: ['(src)/**/*', '!src/core/**/*'],
target: ['src/**/*', '!src/core/**/*'],
from: ['src/core/utils/**/*'],
errorMessage: `Plugins may only import from src/core/server and src/core/public.`,
},
{
target: ['(src)/plugins/*/server/**/*'],
from: ['(src)/plugins/*/public/**/*'],
target: ['src/plugins/*/server/**/*'],
from: ['src/plugins/*/public/**/*'],
errorMessage: `Server code can not import from public, use a common directory.`,
},
{
target: ['(src)/plugins/*/common/**/*'],
from: ['(src)/plugins/*/(server|public)/**/*'],
target: ['src/plugins/*/common/**/*'],
from: ['src/plugins/*/(server|public)/**/*'],
errorMessage: `Common code can not import from server or public, use a common directory.`,
},
{
target: [
'src/legacy/**/*',
'(src)/plugins/**/(public|server)/**/*',
'examples/**/*',
],
target: ['src/legacy/**/*', 'src/plugins/**/(public|server)/**/*', 'examples/**/*'],
from: [
'src/core/public/**/*',
'!src/core/public/index.ts', // relative import
@@ -320,30 +316,30 @@ module.exports = {
{
target: [
'src/legacy/**/*',
'(src)/plugins/**/(public|server)/**/*',
'src/plugins/**/(public|server)/**/*',
'examples/**/*',
'!(src)/**/*.test.*',
'!src/**/*.test.*',
],
from: [
'(src)/plugins/**/(public|server)/**/*',
'!(src)/plugins/**/(public|server)/mocks/index.{js,mjs,ts}',
'!(src)/plugins/**/(public|server)/(index|mocks).{js,mjs,ts,tsx}',
'src/plugins/**/(public|server)/**/*',
'!src/plugins/**/(public|server)/mocks/index.{js,mjs,ts}',
'!src/plugins/**/(public|server)/(index|mocks).{js,mjs,ts,tsx}',
],
allowSameFolder: true,
errorMessage: 'Plugins may only import from top-level public and server modules.',
},
{
target: [
'(src)/plugins/**/*',
'!(src)/plugins/**/server/**/*',
'src/plugins/**/*',
'!src/plugins/**/server/**/*',

'examples/**/*',
'!examples/**/server/**/*',
],
from: [
'src/core/server',
'src/core/server/**/*',
'(src)/plugins/*/server/**/*',
'src/plugins/*/server/**/*',
'examples/**/server/**/*',
],
errorMessage:
@@ -355,7 +351,7 @@ module.exports = {
errorMessage: 'The core cannot depend on any plugins.',
},
{
target: ['(src)/plugins/*/public/**/*'],
target: ['src/plugins/*/public/**/*'],
from: ['ui/**/*'],
errorMessage: 'Plugins cannot import legacy UI code.',
},
@@ -459,7 +455,7 @@ module.exports = {
* Files that run BEFORE node version check
*/
{
files: ['scripts/**/*.js', 'src/setup_node_env/**/*.js'],
files: ['scripts/**/*.js', 'src/setup_node_env/**/!(*.test).js'],
rules: {
'import/no-commonjs': 'off',
'prefer-object-spread/prefer-object-spread': 'off',
@@ -755,5 +751,14 @@ module.exports = {
'no-undef': 'off',
},
},
{
files: [
'src/plugins/data/public/antlr/opensearch_sql/.generated/*',
'src/plugins/data/public/antlr/opensearch_sql/grammar/**/*',
],
rules: {
'filenames/match-regex': 'off',
},
},
],
};
11 changes: 11 additions & 0 deletions .github/.codecov.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
# https://docs.codecov.com/docs/codecov-yaml
codecov:
require_ci_to_pass: yes

coverage:
status:
project:
default:
target: auto
threshold: 2% # the leniency in hitting the target

3 changes: 1 addition & 2 deletions .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
@@ -1,2 +1 @@
# This should match the owning team set up in https://github.com/orgs/opensearch-project/teams
* @opensearch-project/opensearch-dashboards-core
* @ananzh @kavilla @AMoo-Miki @ashwin-pc @joshuarrrr @abbyhu2000 @zengyan-amazon @zhongnansu @manasvinibs @ZilongX @Flyingliuhub @curq @bandinib-amzn @SuZhou-Joe @ruanyl @BionIT @xinruiba @zhyuanqi @mengweieric @LDrago27 @virajsanghvi @sejli @joshuali925
34 changes: 25 additions & 9 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
@@ -1,14 +1,30 @@
### Description
[Describe what this change achieves]


<!-- Describe what this change achieves-->

### Issues Resolved
[List any issues this PR will resolve]


<!-- List any issues this PR will resolve. -->
<!-- Example: closes #1234 -->

## Changelog
<!--
Add a short but concise sentence about the impact of this pull request. Prefix an entry with the type of change they correspond to: breaking, chore, deprecate, doc, feat, fix, infra, refactor, test.
- fix: Update the graph
- feat: Add a new feature
If this change does not need to added to the changelog, just add a single `skip` line e.g.
- skip
Descriptions following the prefixes must be 100 characters long or less
-->

### Check List

- [ ] All tests pass
- [ ] `yarn test:jest`
- [ ] `yarn test:jest_integration`
- [ ] New functionality includes testing.
- [ ] All tests pass
- [ ] `yarn test:jest`
- [ ] `yarn test:jest_integration`
- [ ] `yarn test:ftr`
- [ ] New functionality has been documented.
- [ ] Commits are signed per the DCO using --signoff
- [ ] Update [CHANGELOG.md](./../CHANGELOG.md)
- [ ] Commits are signed per the DCO using --signoff
37 changes: 37 additions & 0 deletions .github/release.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,37 @@
# .github/release.yml

changelog:
exclude:
labels:
- ignore-for-release
authors:
- amazon-auto
categories:
- title: 💥 Breaking Changes
labels:
- breaking
- title: 📈 Features/Enhancements
labels:
- enhancement
- title: 🐛 Bug Fixes
labels:
- bug
- title: 🛡 Security
labels:
- cve
- security vulnerability
- title: 🚞 Infrastructure
labels:
- infra
- title: 📝 Documentation
labels:
- docs
- title: 🛠 Maintenance
labels:
- maintenance
- title: 🪛 Refactoring
labels:
- refactor
- title: 🔩 Tests
labels:
- test
20 changes: 16 additions & 4 deletions .github/workflows/backport.yml
Original file line number Diff line number Diff line change
@@ -12,6 +12,17 @@ jobs:
contents: write
pull-requests: write
name: Backport
# Only react to merged PRs for security reasons.
# See https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#pull_request_target.
if: >
github.event.pull_request.merged
&& (
github.event.action == 'closed'
|| (
github.event.action == 'labeled'
&& contains(github.event.label.name, 'backport')
)
)
steps:
- name: GitHub App token
id: github_app_token
@@ -22,10 +33,11 @@ jobs:
# opensearch-trigger-bot installation ID
installation_id: 22958780

# Using fork of https://github.com/tibdex/backport
# https://github.com/tibdex/backport/pull/81
- name: Backport
uses: VachaShah/backport@v1.1.4
uses: VachaShah/backport@v2.2.0
with:
github_token: ${{ steps.github_app_token.outputs.token }}
branch_name: backport/backport-${{ github.event.number }}
head_template: backport/backport-<%= number %>-to-<%= base %>
files_to_skip: "CHANGELOG.md"
labels_template: "<%= JSON.stringify([...labels, 'autocut']) %>"
failure_labels: "failed backport"
Loading