feat(AnalyticalTable): add onSort callback to Table #123
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
MarcusNotheis
approved these changes
Sep 17, 2019
renovate bot
added a commit
that referenced
this pull request
Feb 5, 2024
[](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [@babel/cli](https://babel.dev/docs/en/next/babel-cli) ([source](https://togithub.com/babel/babel/tree/HEAD/packages/babel-cli)) | [`7.23.4` -> `7.23.9`](https://renovatebot.com/diffs/npm/@babel%2fcli/7.23.4/7.23.9) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | | [@babel/core](https://babel.dev/docs/en/next/babel-core) ([source](https://togithub.com/babel/babel/tree/HEAD/packages/babel-core)) | [`7.23.7` -> `7.23.9`](https://renovatebot.com/diffs/npm/@babel%2fcore/7.23.7/7.23.9) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | | [@babel/preset-env](https://babel.dev/docs/en/next/babel-preset-env) ([source](https://togithub.com/babel/babel/tree/HEAD/packages/babel-preset-env)) | [`7.23.8` -> `7.23.9`](https://renovatebot.com/diffs/npm/@babel%2fpreset-env/7.23.8/7.23.9) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | | [@cypress/code-coverage](https://togithub.com/cypress-io/code-coverage) | [`3.12.18` -> `3.12.19`](https://renovatebot.com/diffs/npm/@cypress%2fcode-coverage/3.12.18/3.12.19) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | | [@storybook/addon-a11y](https://togithub.com/storybookjs/storybook/tree/next/code/addons/a11y) ([source](https://togithub.com/storybookjs/storybook/tree/HEAD/code/addons/a11y)) | [`7.6.10` -> `7.6.12`](https://renovatebot.com/diffs/npm/@storybook%2faddon-a11y/7.6.10/7.6.12) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | | [@storybook/addon-essentials](https://togithub.com/storybookjs/storybook/tree/next/code/addons/essentials) ([source](https://togithub.com/storybookjs/storybook/tree/HEAD/code/addons/essentials)) | [`7.6.10` -> `7.6.12`](https://renovatebot.com/diffs/npm/@storybook%2faddon-essentials/7.6.10/7.6.12) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | | [@storybook/blocks](https://togithub.com/storybookjs/storybook/tree/next/code/ui/blocks) ([source](https://togithub.com/storybookjs/storybook/tree/HEAD/code/ui/blocks)) | [`7.6.10` -> `7.6.12`](https://renovatebot.com/diffs/npm/@storybook%2fblocks/7.6.10/7.6.12) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | | [@storybook/react](https://togithub.com/storybookjs/storybook/tree/next/code/renderers/react) ([source](https://togithub.com/storybookjs/storybook/tree/HEAD/code/renderers/react)) | [`7.6.10` -> `7.6.12`](https://renovatebot.com/diffs/npm/@storybook%2freact/7.6.10/7.6.12) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | | [@storybook/react-vite](https://togithub.com/storybookjs/storybook/tree/next/code/frameworks/react-vite) ([source](https://togithub.com/storybookjs/storybook/tree/HEAD/code/frameworks/react-vite)) | [`7.6.10` -> `7.6.12`](https://renovatebot.com/diffs/npm/@storybook%2freact-vite/7.6.10/7.6.12) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | | [@storybook/theming](https://togithub.com/storybookjs/storybook/tree/next/code/lib/theming) ([source](https://togithub.com/storybookjs/storybook/tree/HEAD/code/lib/theming)) | [`7.6.10` -> `7.6.12`](https://renovatebot.com/diffs/npm/@storybook%2ftheming/7.6.10/7.6.12) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | | [@types/node](https://togithub.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/node) ([source](https://togithub.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node)) | [`20.11.5` -> `20.11.16`](https://renovatebot.com/diffs/npm/@types%2fnode/20.11.5/20.11.16) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | | [@types/react](https://togithub.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/react) ([source](https://togithub.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/react)) | [`18.2.48` -> `18.2.53`](https://renovatebot.com/diffs/npm/@types%2freact/18.2.48/18.2.53) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | | [cypress](https://cypress.io) ([source](https://togithub.com/cypress-io/cypress)) | [`13.6.3` -> `13.6.4`](https://renovatebot.com/diffs/npm/cypress/13.6.3/13.6.4) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | | [husky](https://togithub.com/typicode/husky) | [`9.0.6` -> `9.0.10`](https://renovatebot.com/diffs/npm/husky/9.0.6/9.0.10) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | | [lint-staged](https://togithub.com/okonet/lint-staged) | [`15.2.0` -> `15.2.1`](https://renovatebot.com/diffs/npm/lint-staged/15.2.0/15.2.1) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | | [npm-run-all2](https://togithub.com/bcomnes/npm-run-all2) | [`6.1.1` -> `6.1.2`](https://renovatebot.com/diffs/npm/npm-run-all2/6.1.1/6.1.2) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | | [prettier](https://prettier.io) ([source](https://togithub.com/prettier/prettier)) | [`3.2.4` -> `3.2.5`](https://renovatebot.com/diffs/npm/prettier/3.2.4/3.2.5) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | | [storybook](https://togithub.com/storybookjs/storybook/tree/next/code/lib/cli) ([source](https://togithub.com/storybookjs/storybook/tree/HEAD/code/lib/cli)) | [`7.6.10` -> `7.6.12`](https://renovatebot.com/diffs/npm/storybook/7.6.10/7.6.12) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | [](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>babel/babel (@​babel/cli)</summary> ### [`v7.23.9`](https://togithub.com/babel/babel/blob/HEAD/CHANGELOG.md#v7239-2024-01-25) [Compare Source](https://togithub.com/babel/babel/compare/v7.23.4...v7.23.9) ##### 🐛 Bug Fix - `babel-helper-transform-fixture-test-runner`, `babel-plugin-transform-function-name`, `babel-plugin-transform-modules-systemjs`, `babel-preset-env` - [#​16225](https://togithub.com/babel/babel/pull/16225) fix: `systemjs` re-traverses helpers ([@​liuxingbaoyu](https://togithub.com/liuxingbaoyu)) - `babel-helper-create-class-features-plugin`, `babel-plugin-proposal-decorators` - [#​16226](https://togithub.com/babel/babel/pull/16226) Improve decorated private method check ([@​JLHwung](https://togithub.com/JLHwung)) - `babel-plugin-proposal-decorators`, `babel-plugin-transform-async-generator-functions`, `babel-plugin-transform-runtime`, `babel-preset-env` - [#​16224](https://togithub.com/babel/babel/pull/16224) Properly sort `core-js@3` imports ([@​nicolo-ribaudo](https://togithub.com/nicolo-ribaudo)) - `babel-traverse` - [#​15383](https://togithub.com/babel/babel/pull/15383) fix: Don't throw in `getTypeAnnotation` when using TS+inference ([@​liuxingbaoyu](https://togithub.com/liuxingbaoyu)) - Other - [#​16210](https://togithub.com/babel/babel/pull/16210) \[eslint] Fix `no-use-before-define` for class ref in fields ([@​nicolo-ribaudo](https://togithub.com/nicolo-ribaudo)) ##### 🏠 Internal - `babel-core`, `babel-parser`, `babel-template` - [#​16222](https://togithub.com/babel/babel/pull/16222) Migrate `eslint-parser` to cts ([@​liuxingbaoyu](https://togithub.com/liuxingbaoyu)) - `babel-types` - [#​16213](https://togithub.com/babel/babel/pull/16213) Remove `@babel/types` props that are not produced by the parser ([@​liuxingbaoyu](https://togithub.com/liuxingbaoyu)) ##### 🏃♀️ Performance - `babel-parser` - [#​16072](https://togithub.com/babel/babel/pull/16072) perf: Improve parser performance for typescript ([@​liuxingbaoyu](https://togithub.com/liuxingbaoyu)) ##### 🔬 Output optimization - `babel-helper-create-class-features-plugin`, `babel-plugin-proposal-decorators`, `babel-plugin-proposal-destructuring-private`, `babel-plugin-proposal-pipeline-operator`, `babel-plugin-transform-class-properties`, `babel-plugin-transform-class-static-block`, `babel-plugin-transform-new-target`, `babel-plugin-transform-parameters`, `babel-plugin-transform-private-methods`, `babel-preset-env` - [#​16218](https://togithub.com/babel/babel/pull/16218) Improve temporary variables for decorators ([@​liuxingbaoyu](https://togithub.com/liuxingbaoyu)) - `babel-helpers`, `babel-plugin-proposal-explicit-resource-management`, `babel-runtime-corejs2`, `babel-runtime-corejs3`, `babel-runtime` - [#​15959](https://togithub.com/babel/babel/pull/15959) Improve output of `using` ([@​liuxingbaoyu](https://togithub.com/liuxingbaoyu)) </details> <details> <summary>cypress-io/code-coverage (@​cypress/code-coverage)</summary> ### [`v3.12.19`](https://togithub.com/cypress-io/code-coverage/releases/tag/v3.12.19) [Compare Source](https://togithub.com/cypress-io/code-coverage/compare/v3.12.18...v3.12.19) ##### Bug Fixes - **deps:** update dependency [@​types/react](https://togithub.com/types/react) to v18.2.48 ([#​769](https://togithub.com/cypress-io/code-coverage/issues/769)) ([b87408b](https://togithub.com/cypress-io/code-coverage/commit/b87408ba2d5a10958c8934a5ed08c9a9992ccfb5)) </details> <details> <summary>storybookjs/storybook (@​storybook/addon-a11y)</summary> ### [`v7.6.12`](https://togithub.com/storybookjs/storybook/blob/HEAD/CHANGELOG.md#7612) [Compare Source](https://togithub.com/storybookjs/storybook/compare/v7.6.11...v7.6.12) - CLI: Fix `upgrade` detecting the wrong version of existing Storybooks - [#​25752](https://togithub.com/storybookjs/storybook/pull/25752), thanks [@​JReinhold](https://togithub.com/JReinhold)! ### [`v7.6.11`](https://togithub.com/storybookjs/storybook/blob/HEAD/CHANGELOG.md#7611) [Compare Source](https://togithub.com/storybookjs/storybook/compare/v7.6.10...v7.6.11) - CLI: Update init for react native v7 - [#​25780](https://togithub.com/storybookjs/storybook/pull/25780), thanks [@​dannyhw](https://togithub.com/dannyhw)! - Codemods: Add support for multiple file extensions in runCodemod function - [#​25708](https://togithub.com/storybookjs/storybook/pull/25708), thanks [@​valentinpalkovic](https://togithub.com/valentinpalkovic)! </details> <details> <summary>cypress-io/cypress (cypress)</summary> ### [`v13.6.4`](https://togithub.com/cypress-io/cypress/releases/tag/v13.6.4) [Compare Source](https://togithub.com/cypress-io/cypress/compare/v13.6.3...v13.6.4) Changelog: https://docs.cypress.io/guides/references/changelog#13-6-4 </details> <details> <summary>typicode/husky (husky)</summary> ### [`v9.0.10`](https://togithub.com/typicode/husky/releases/tag/v9.0.10) [Compare Source](https://togithub.com/typicode/husky/compare/v9.0.9...v9.0.10) - fix: rename index.d.ts to index.d.mts by [@​mrkjdy](https://togithub.com/mrkjdy) in [https://github.com/typicode/husky/pull/1379](https://togithub.com/typicode/husky/pull/1379) ### [`v9.0.9`](https://togithub.com/typicode/husky/releases/tag/v9.0.9) [Compare Source](https://togithub.com/typicode/husky/compare/v9.0.8...v9.0.9) - refactor: rename files by [@​typicode](https://togithub.com/typicode) in [https://github.com/typicode/husky/pull/1378](https://togithub.com/typicode/husky/pull/1378) ### [`v9.0.8`](https://togithub.com/typicode/husky/releases/tag/v9.0.8) [Compare Source](https://togithub.com/typicode/husky/compare/v9.0.7...v9.0.8) - docs: update index.md by [@​khaledYS](https://togithub.com/khaledYS) in [https://github.com/typicode/husky/pull/1369](https://togithub.com/typicode/husky/pull/1369) - Fix tab detection on install command by [@​glensc](https://togithub.com/glensc) in [https://github.com/typicode/husky/pull/1376](https://togithub.com/typicode/husky/pull/1376) - refactor: reduce file size by [@​typicode](https://togithub.com/typicode) in [https://github.com/typicode/husky/pull/1377](https://togithub.com/typicode/husky/pull/1377) ### [`v9.0.7`](https://togithub.com/typicode/husky/releases/tag/v9.0.7) [Compare Source](https://togithub.com/typicode/husky/compare/v9.0.6...v9.0.7) - fix: typo and source `~/.huskyrc` correctly (compatibility with v8) - docs: fix example by [@​typicode](https://togithub.com/typicode) in [https://github.com/typicode/husky/pull/1363](https://togithub.com/typicode/husky/pull/1363) </details> <details> <summary>okonet/lint-staged (lint-staged)</summary> ### [`v15.2.1`](https://togithub.com/okonet/lint-staged/blob/HEAD/CHANGELOG.md#1521) [Compare Source](https://togithub.com/okonet/lint-staged/compare/v15.2.0...v15.2.1) ##### Patch Changes - [#​1387](https://togithub.com/lint-staged/lint-staged/pull/1387) [`e4023f6`](https://togithub.com/lint-staged/lint-staged/commit/e4023f687616dcf4816545b8eefbcce50e255c9c) Thanks [@​iiroj](https://togithub.com/iiroj)! - Ignore stdin of spawned commands so that they don't get stuck waiting. Until now, *lint-staged* has used the default settings to spawn linter commands. This means the `stdin` of the spawned commands has accepted input, and essentially gotten stuck waiting. Now the `stdin` is ignored and commands will no longer get stuck. If you relied on this behavior, please open a new issue and describe how; the behavior has not been intended. </details> <details> <summary>bcomnes/npm-run-all2 (npm-run-all2)</summary> ### [`v6.1.2`](https://togithub.com/bcomnes/npm-run-all2/blob/HEAD/CHANGELOG.md#v612) [Compare Source](https://togithub.com/bcomnes/npm-run-all2/compare/v6.1.1...v6.1.2) ##### Merged - feat: move to read-package-json-fast [`#130`](https://togithub.com/bcomnes/npm-run-all2/pull/130) - Upgrade: Bump c8 from 8.0.1 to 9.0.0 [`#127`](https://togithub.com/bcomnes/npm-run-all2/pull/127) - Upgrade: Bump github/codeql-action from 2 to 3 [`#126`](https://togithub.com/bcomnes/npm-run-all2/pull/126) - Upgrade: Bump actions/setup-node from 3 to 4 [`#123`](https://togithub.com/bcomnes/npm-run-all2/pull/123) ##### Commits - Publish the whole project [`3dde20c`](https://togithub.com/bcomnes/npm-run-all2/commit/3dde20c1c8fa973045773e03f4fc121360fdbed4) - Utilize CJS require for 'read-package-json-fast' [`605ca15`](https://togithub.com/bcomnes/npm-run-all2/commit/605ca15d9adee3ce14da6fcaa98cb14d9c03795c) - Update FUNDING.yml [`c838ee9`](https://togithub.com/bcomnes/npm-run-all2/commit/c838ee9eea06e545d1a7f25592f7beb8468f1afd) </details> <details> <summary>prettier/prettier (prettier)</summary> ### [`v3.2.5`](https://togithub.com/prettier/prettier/blob/HEAD/CHANGELOG.md#325) [Compare Source](https://togithub.com/prettier/prettier/compare/3.2.4...3.2.5) [diff](https://togithub.com/prettier/prettier/compare/3.2.4...3.2.5) ##### Support Angular inline styles as single template literal ([#​15968](https://togithub.com/prettier/prettier/pull/15968) by [@​sosukesuzuki](https://togithub.com/sosukesuzuki)) [Angular v17](https://blog.angular.io/introducing-angular-v17-4d7033312e4b) supports single string inline styles. <!-- prettier-ignore --> ```ts // Input @​Component({ template: `<div>...</div>`, styles: `h1 { color: blue; }`, }) export class AppComponent {} // Prettier 3.2.4 @​Component({ template: `<div>...</div>`, styles: `h1 { color: blue; }`, }) export class AppComponent {} // Prettier 3.2.5 @​Component({ template: `<div>...</div>`, styles: ` h1 { color: blue; } `, }) export class AppComponent {} ``` ##### Unexpected embedded formatting for Angular template ([#​15969](https://togithub.com/prettier/prettier/pull/15969) by [@​JounQin](https://togithub.com/JounQin)) Computed template should not be considered as Angular component template <!-- prettier-ignore --> ```ts // Input const template = "foobar"; @​Component({ [template]: `<h1>{{ hello }}</h1>`, }) export class AppComponent {} // Prettier 3.2.4 const template = "foobar"; @​Component({ [template]: `<h1>{{ hello }}</h1>`, }) export class AppComponent {} // Prettier 3.2.5 const template = "foobar"; @​Component({ [template]: `<h1>{{ hello }}</h1>`, }) export class AppComponent {} ``` ##### Use `"json"` parser for `tsconfig.json` by default ([#​16012](https://togithub.com/prettier/prettier/pull/16012) by [@​sosukesuzuki](https://togithub.com/sosukesuzuki)) In [v2.3.0](https://prettier.io/blog/2024/01/12/3.2.0#new-jsonc-parser-added-15831httpsgithubcomprettierprettierpull15831-by-fiskerhttpsgithubcomfisker), we introduced `"jsonc"` parser which adds trialing comma **by default**. When adding a new parser we also define how it will be used based on the [`linguist-languages`](https://www.npmjs.com/package/linguist-languages) data. `tsconfig.json` is a special file used by [TypeScript](https://www.typescriptlang.org/docs/handbook/tsconfig-json.html#using-tsconfigjson-or-jsconfigjson), it uses `.json` file extension, but it actually uses the [JSON with Comments](https://code.visualstudio.com/docs/languages/json#\_json-with-comments) syntax. However, we found that there are many third-party tools not recognize it correctly because of the confusing `.json` file extension. We decide to treat it as a JSON file for now to avoid the extra configuration step. To keep using the `"jsonc"` parser for your `tsconfig.json` files, add the following to your `.pretterrc` file ```json { "overrides": [ { "files": ["tsconfig.json", "jsconfig.json"], "options": { "parser": "jsonc" } } ] } ``` <!-- prettier-ignore --> ``` ``` </details> --- ### Configuration 📅 **Schedule**: Branch creation - "before 4am on Monday" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Enabled. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 👻 **Immortal**: This PR will be recreated if closed unmerged. Get [config help](https://togithub.com/renovatebot/renovate/discussions) if that's undesired. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/SAP/ui5-webcomponents-react). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4xNzAuMCIsInVwZGF0ZWRJblZlciI6IjM3LjE3MC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9--> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
](https://renovatebot.com){kind=link}
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.