Android 1.31.X + 1.32.X, iOS 2.15.0 (pending) #99
Draft
mikimandoki wants to merge 193 commits intomainfrom
Draft
Android 1.31.X + 1.32.X, iOS 2.15.0 (pending) #99mikimandoki wants to merge 193 commits intomainfrom
mikimandoki wants to merge 193 commits intomainfrom
Conversation
feat: pro tests
Bumps the monthly-updates group with 15 updates: | Package | From | To | | --- | --- | --- | | [appium-uiautomator2-driver](https://github.com/appium/appium-uiautomator2-driver) | `6.8.0` | `7.0.0` | | [appium-xcuitest-driver](https://github.com/appium/appium-xcuitest-driver) | `10.21.2` | `10.24.1` | | [dotenv](https://github.com/motdotla/dotenv) | `17.2.4` | `17.3.1` | | [@types/lodash](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/lodash) | `4.17.23` | `4.17.24` | | [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node) | `25.2.3` | `25.3.3` | | [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin) | `8.55.0` | `8.56.1` | | [@typescript-eslint/parser](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/parser) | `8.55.0` | `8.56.1` | | [allure-commandline](https://github.com/allure-framework/allure-npm) | `2.36.0` | `2.37.0` | | [allure-js-commons](https://github.com/allure-framework/allure-js/tree/HEAD/packages/allure-js-commons) | `3.4.5` | `3.5.0` | | [eslint](https://github.com/eslint/eslint) | `10.0.0` | `10.0.2` | | [eslint-plugin-perfectionist](https://github.com/azat-io/eslint-plugin-perfectionist) | `5.5.0` | `5.6.0` | | [glob](https://github.com/isaacs/node-glob) | `13.0.2` | `13.0.6` | | [globals](https://github.com/sindresorhus/globals) | `17.3.0` | `17.4.0` | | [typescript-eslint](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/typescript-eslint) | `8.55.0` | `8.56.1` | | [undici](https://github.com/nodejs/undici) | `7.21.0` | `7.22.0` | Updates `appium-uiautomator2-driver` from 6.8.0 to 7.0.0 - [Release notes](https://github.com/appium/appium-uiautomator2-driver/releases) - [Changelog](https://github.com/appium/appium-uiautomator2-driver/blob/master/CHANGELOG.md) - [Commits](appium/appium-uiautomator2-driver@v6.8.0...v7.0.0) Updates `appium-xcuitest-driver` from 10.21.2 to 10.24.1 - [Release notes](https://github.com/appium/appium-xcuitest-driver/releases) - [Changelog](https://github.com/appium/appium-xcuitest-driver/blob/master/CHANGELOG.md) - [Commits](appium/appium-xcuitest-driver@v10.21.2...v10.24.1) Updates `dotenv` from 17.2.4 to 17.3.1 - [Changelog](https://github.com/motdotla/dotenv/blob/master/CHANGELOG.md) - [Commits](motdotla/dotenv@v17.2.4...v17.3.1) Updates `@types/lodash` from 4.17.23 to 4.17.24 - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/lodash) Updates `@types/node` from 25.2.3 to 25.3.3 - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/node) Updates `@typescript-eslint/eslint-plugin` from 8.55.0 to 8.56.1 - [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases) - [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/CHANGELOG.md) - [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.56.1/packages/eslint-plugin) Updates `@typescript-eslint/parser` from 8.55.0 to 8.56.1 - [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases) - [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/parser/CHANGELOG.md) - [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.56.1/packages/parser) Updates `allure-commandline` from 2.36.0 to 2.37.0 - [Release notes](https://github.com/allure-framework/allure-npm/releases) - [Commits](allure-framework/allure-npm@2.36.0...2.37.0) Updates `allure-js-commons` from 3.4.5 to 3.5.0 - [Release notes](https://github.com/allure-framework/allure-js/releases) - [Commits](https://github.com/allure-framework/allure-js/commits/v3.5.0/packages/allure-js-commons) Updates `eslint` from 10.0.0 to 10.0.2 - [Release notes](https://github.com/eslint/eslint/releases) - [Commits](eslint/eslint@v10.0.0...v10.0.2) Updates `eslint-plugin-perfectionist` from 5.5.0 to 5.6.0 - [Release notes](https://github.com/azat-io/eslint-plugin-perfectionist/releases) - [Changelog](https://github.com/azat-io/eslint-plugin-perfectionist/blob/main/changelog.md) - [Commits](azat-io/eslint-plugin-perfectionist@v5.5.0...v5.6.0) Updates `glob` from 13.0.2 to 13.0.6 - [Changelog](https://github.com/isaacs/node-glob/blob/main/changelog.md) - [Commits](isaacs/node-glob@v13.0.2...v13.0.6) Updates `globals` from 17.3.0 to 17.4.0 - [Release notes](https://github.com/sindresorhus/globals/releases) - [Commits](sindresorhus/globals@v17.3.0...v17.4.0) Updates `typescript-eslint` from 8.55.0 to 8.56.1 - [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases) - [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/typescript-eslint/CHANGELOG.md) - [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.56.1/packages/typescript-eslint) Updates `undici` from 7.21.0 to 7.22.0 - [Release notes](https://github.com/nodejs/undici/releases) - [Commits](nodejs/undici@v7.21.0...v7.22.0) --- updated-dependencies: - dependency-name: appium-uiautomator2-driver dependency-version: 7.0.0 dependency-type: direct:production update-type: version-update:semver-major dependency-group: monthly-updates - dependency-name: appium-xcuitest-driver dependency-version: 10.24.1 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: monthly-updates - dependency-name: dotenv dependency-version: 17.3.1 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: monthly-updates - dependency-name: "@types/lodash" dependency-version: 4.17.24 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: monthly-updates - dependency-name: "@types/node" dependency-version: 25.3.3 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: monthly-updates - dependency-name: "@typescript-eslint/eslint-plugin" dependency-version: 8.56.1 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: monthly-updates - dependency-name: "@typescript-eslint/parser" dependency-version: 8.56.1 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: monthly-updates - dependency-name: allure-commandline dependency-version: 2.37.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: monthly-updates - dependency-name: allure-js-commons dependency-version: 3.5.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: monthly-updates - dependency-name: eslint dependency-version: 10.0.2 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: monthly-updates - dependency-name: eslint-plugin-perfectionist dependency-version: 5.6.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: monthly-updates - dependency-name: glob dependency-version: 13.0.6 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: monthly-updates - dependency-name: globals dependency-version: 17.4.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: monthly-updates - dependency-name: typescript-eslint dependency-version: 8.56.1 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: monthly-updates - dependency-name: undici dependency-version: 7.22.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: monthly-updates ... Signed-off-by: dependabot[bot] <support@github.com>
…rn/dev/monthly-updates-4da85c507d chore(deps): bump the monthly-updates group with 15 updates
There was a problem hiding this comment.
Pull request overview
Updates the mobile QA automation harness to support newer Android/iOS builds, adds Session Pro/CTA coverage, improves failure artifact collection, and hardens CI emulator and workflow behavior.
Changes:
- Added Android emulator recovery tooling and CI workflow hooks to self-heal mid-job emulator failures.
- Introduced Session Pro/CTA infrastructure (CTA configs,
@protagging, new Pro-focused tests) and expanded community helpers/constants. - Improved test diagnostics: cleaner Allure output, failure screenshots + per-device logs, and a simplified
assert()wrapper.
Reviewed changes
Copilot reviewed 133 out of 137 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| tsconfig.json | Bumps TS compilation target to ES2022. |
| scripts/emulator_health.ts | Adds script to detect/recover missing Android emulators in CI. |
| scripts/create_ios_simulators.ts | Adds animated GIF to iOS media seeding. |
| run/types/tuple.d.ts | Removes tuple type helper definitions. |
| run/types/sessionIt.ts | Adds isPro tagging and captures logs on failure. |
| run/types/cta.ts | Introduces CTA config types and centralized CTA strings. |
| run/types/allure.ts | Expands Allure suites and adds new test step labels. |
| run/test/utils/utilities.ts | Adds assert() wrapper and ensures restart returns to home screen. |
| run/test/utils/test_setup.ts | Removes old manual setup comment block. |
| run/test/utils/set_disappearing_messages.ts | Simplifies disappearing-message helper signature/behavior. |
| run/test/utils/screenshot_helper.ts | Adds log capture on failure and switches to shared device registry. |
| run/test/utils/restore_account.ts | Adds explicit deviceIdentity and updates permissions handler. |
| run/test/utils/permissions.ts | Splits notification vs background permissions handling. |
| run/test/utils/open_app.ts | Adds emulator recovery path in CI; async device registry registration; uninstall helper. |
| run/test/utils/link_device.ts | Updates to new notification-permissions handler. |
| run/test/utils/join_community.ts | Removes old join community helper (moved). |
| run/test/utils/index.ts | Exports new assert() helper. |
| run/test/utils/handle_first_open.ts | Minor cleanup of first-open Photos flow. |
| run/test/utils/get_account_id.ts | Documents sort helper intent. |
| run/test/utils/disappearing_control_messages.ts | Removes old control-message helper (inlined in tests). |
| run/test/utils/device_registry.ts | Adds per-test device registry with log context (PID/timestamps). |
| run/test/utils/create_group.ts | Improves group verification by sending/validating messages across all devices. |
| run/test/utils/create_contact.ts | Simplifies message-request acceptance and reply message content. |
| run/test/utils/create_account.ts | Adds slow-mode support and background permissions handling; updates recovery phrase path. |
| run/test/utils/conversation_order.ts | Adds helpers to assert pinned conversation ordering. |
| run/test/utils/community.ts | Adds community join helpers and admin-seed gating for ban tests. |
| run/test/utils/click_by_coordinates.ts | Removes fixed sleep before coordinate tap. |
| run/test/utils/capabilities_ios.ts | Adds iOS test context (Pro flag), increases command timeout, disables animations. |
| run/test/utils/capabilities_android.ts | Reduces emulator UDID list and removes redundant capability aliases. |
| run/test/utils/binaries.ts | Removes unused Android SDK manager/AVD manager/system-image helpers. |
| run/test/utils/allure/allureHelpers.ts | Adds network target to Allure environment properties. |
| run/test/state_builder/index.ts | Converts seeded users to User type and threads iOS context through app open. |
| run/test/specs/voice_calls.spec.ts | Updates enable/notification modal locators. |
| run/test/specs/user_actions_unblock_user.spec.ts | Adds iOS scroll workaround before block/unblock actions. |
| run/test/specs/user_actions_share_to_session.spec.ts | Improves test coverage for Android-only “share within Session” flow. |
| run/test/specs/user_actions_pin_unpin.spec.ts | Adds pin/unpin tests including Pro vs non-Pro pin limits. |
| run/test/specs/user_actions_delete_contact_ucs.spec.ts | Updates message request navigation to new locator. |
| run/test/specs/user_actions_create_contact.spec.ts | Uses unified message-request acceptance helper. |
| run/test/specs/user_actions_change_username.spec.ts | Removes outdated Allure link metadata. |
| run/test/specs/user_actions_block_conversation_options.spec.ts | Adds iOS scroll workaround before blocking. |
| run/test/specs/user_actions_animated_profile_picture.spec.ts | Adds animated profile picture and Pro CTA coverage. |
| run/test/specs/upm_homescreen.spec.ts | Fixes seeded user field usage (accountID). |
| run/test/specs/slow_mode_background.spec.ts | Adds Android slow-mode background permissions modal test + uninstall cleanup. |
| run/test/specs/review_positive.spec.ts | Updates rate modal assertions for new single-button UI. |
| run/test/specs/recovery_banner.spec.ts | Adds Android recovery password banner behavior tests. |
| run/test/specs/network_page_refresh_page.spec.ts | Removes outdated Allure link metadata. |
| run/test/specs/message_voice.spec.ts | Uses offset long-press to avoid scrubber playback. |
| run/test/specs/message_requests_delete.spec.ts | Updates iOS swipe target to new locator. |
| run/test/specs/message_requests_decline.spec.ts | Updates message request navigation to new locator. |
| run/test/specs/message_requests_block.spec.ts | Updates message request navigation to new locator. |
| run/test/specs/message_requests_accept_text_reply.spec.ts | Updates message request navigation to new locator. |
| run/test/specs/message_requests_accept.spec.ts | Uses unified message-request acceptance helper. |
| run/test/specs/message_length.spec.ts | Adds Pro vs non-Pro message length cases + CTA/modal expectations. |
| run/test/specs/message_community_invitation.spec.ts | Migrates to centralized community constants/helpers. |
| run/test/specs/linked_group_leave.spec.ts | Improves linked-device group leave verification logic. |
| run/test/specs/linked_device_restore_group.spec.ts | Updates restore flow to set deviceIdentity. |
| run/test/specs/linked_device_community_ban.spec.ts | Adds linked-device community ban/unban coverage. |
| run/test/specs/linked_device_block_user.spec.ts | Simplifies and hardens blocked-user verification. |
| run/test/specs/group_tests_kick_member_messages.spec.ts | Adds Android-only kick + delete messages test. |
| run/test/specs/group_tests_kick_member.spec.ts | Refactors kick-member test steps and improves assertions; adds closeApp. |
| run/test/specs/group_tests_invite_contact_banner.spec.ts | Changes expectation to banner absence. |
| run/test/specs/group_tests_edit_group_banner.spec.ts | Changes expectation to banner absence; updates description. |
| run/test/specs/group_tests_delete_group.spec.ts | Improves empty-state locator reuse + adds Allure link. |
| run/test/specs/group_tests_create_group_banner.spec.ts | Changes expectation to banner absence; updates description. |
| run/test/specs/group_tests_admin_leave_group.spec.ts | Adds Android admin leave/delete group behavior tests. |
| run/test/specs/group_tests_add_contact_nohistory.spec.ts | Adds “invite without history” group test. |
| run/test/specs/group_tests_add_contact.spec.ts | Adds Android “invite with history” test flow and assertions. |
| run/test/specs/group_tests_add_accountid.spec.ts | Adds Android “invite Account ID (non-contact)” group test. |
| run/test/specs/group_message_voice.spec.ts | Fixes wording and long-press behavior for voice messages. |
| run/test/specs/group_disappearing_messages_voice.spec.ts | Updates helper call signature. |
| run/test/specs/group_disappearing_messages_video.spec.ts | Updates helper call signature. |
| run/test/specs/group_disappearing_messages_link.spec.ts | Updates helper call signature. |
| run/test/specs/group_disappearing_messages_image.spec.ts | Updates helper call signature. |
| run/test/specs/group_disappearing_messages_gif.spec.ts | Updates helper call signature. |
| run/test/specs/disappearing_voice.spec.ts | Updates helper call signature. |
| run/test/specs/disappearing_video.spec.ts | Updates helper call signature. |
| run/test/specs/disappearing_messages_follow_settings.spec.ts | Adds new follow-setting coverage for 1:1. |
| run/test/specs/disappearing_messages_defaults.spec.ts | Adds default-timer selection coverage across convo types. |
| run/test/specs/disappearing_link.spec.ts | Updates helper call signature. |
| run/test/specs/disappearing_image.spec.ts | Updates helper call signature. |
| run/test/specs/disappearing_gif.spec.ts | Updates helper call signature. |
| run/test/specs/disappearing_community_invite.spec.ts | Migrates to centralized communities + adjusts back navigation per platform. |
| run/test/specs/disappearing_call.spec.ts | Updates helper call signature and modal locators; adds Allure link. |
| run/test/specs/disappear_after_send_off_1o1.spec.ts | Inlines control-message checks and removes follow-setting flow. |
| run/test/specs/disappear_after_send_note_to_self.spec.ts | Updates helper call signature. |
| run/test/specs/disappear_after_send_groups.spec.ts | Updates helper call signature. |
| run/test/specs/disappear_after_send.spec.ts | Inlines control-message checks and updates helper call signature. |
| run/test/specs/disappear_after_read.spec.ts | Inlines control-message checks and updates helper call signature. |
| run/test/specs/cta_donate_time.spec.ts | Switches to checkCTA() and verifyNoCTAShows() helpers. |
| run/test/specs/cta_donate_review.spec.ts | Switches to checkCTA() and renames restart helper import. |
| run/test/specs/community_tests_join.spec.ts | Migrates to centralized communities + improves step naming. |
| run/test/specs/community_tests_image.spec.ts | Migrates to centralized communities + minor trust-attachments update. |
| run/test/specs/community_requests_on.spec.ts | Migrates to centralized communities and updates message request navigation. |
| run/test/specs/community_requests_off.spec.ts | Migrates to centralized communities. |
| run/test/specs/community_emoji_react.spec.ts | Migrates to centralized communities and adjusts steps. |
| run/test/specs/community_ban.spec.ts | Adds community admin ban/unban and ban+delete coverage. |
| run/test/specs/app_disguise_set.spec.ts | Uses shared uninstall helper and fixes modal step label spelling. |
| run/test/media/animated_profile_picture.gif | Adds animated GIF test asset (LFS). |
| run/test/locators/settings.ts | Adds settings modal enable locator, Pro animated description locator, and scrollable selectors. |
| run/test/locators/onboarding.ts | Updates iOS link selectors for policy/ToS buttons. |
| run/test/locators/index.ts | Adds GIFName locator, InviteAccountIDOrONS locator, improves locator description truncation. |
| run/test/locators/home.ts | Adds background permissions locators and pin/unpin locators. |
| run/test/locators/groups.ts | Adds/manage-admins & invite-history locators; updates some selectors and iOS delete group label. |
| run/test/locators/global.ts | Updates CTA locators to use accessibility IDs and adds iOS CTA feature indexing. |
| run/test/locators/conversation.ts | Adds accept message request + ban/unban context menu locators; migrates community name lookup. |
| run/screenshots/ios/cta_pro_activated.png | Adds iOS screenshot baseline (LFS). |
| run/screenshots/android/settings_privacy.png | Updates Android screenshot baseline (LFS). |
| run/screenshots/android/settings_notifications.png | Updates Android screenshot baseline (LFS). |
| run/screenshots/android/settings_conversations.png | Updates Android screenshot baseline (LFS). |
| run/screenshots/android/settings.png | Updates Android screenshot baseline (LFS). |
| run/screenshots/android/landingpage_new_account.png | Updates Android screenshot baseline (LFS). |
| run/screenshots/android/cta_pro_activated.png | Adds Android screenshot baseline (LFS). |
| run/screenshots/android/cta_donate.png | Updates Android screenshot baseline (LFS). |
| run/localizer/lib | Updates localizer submodule pointer. |
| run/localizer/englishStrippedStr.ts | Removes english-only stripped string helper. |
| run/constants/testfiles.ts | Adds animated profile picture testfile constant. |
| run/constants/index.ts | Adds Pro backend URL constant. |
| run/constants/community.ts | Replaces single test community values with a community registry. |
| playwright.config.ts | Disables Allure Playwright internal step detail. |
| patches/allure-playwright@3.4.5.patch | Renames Allure stdout/stderr attachments for clarity. |
| package.json | Adds recover-emulators script, updates packages, and enables patched allure-playwright dependency. |
| github/actions/setup/action.yml | Forces IPv4 DNS result order during dependency install. |
| github/actions/list-tests/action.yml | Adds optional grep-invert flag support for @pro filtering. |
| github/actions/generate-publish-test-report/action.yml | Threads NETWORK_TARGET into report generation/publish steps. |
| eslint.config.mjs | Migrates config export to defineConfig. |
| appium_next.d.ts | Removes custom Appium “next” typings file. |
| README.md | Updates “test-one” usage instructions. |
| .github/workflows/ios-regression.yml | Adds RUN_PRO_TESTS toggle, permissions, and NETWORK_TARGET; supports grep-invert. |
| .github/workflows/android-regression.yml | Adds NETWORK_TARGET input, RUN_PRO_TESTS toggle, emulator recovery, and report network metadata. |
| .gitattributes | Expands LFS tracking to common media extensions. |
| .env.sample | Removes obsolete Android SDK manager env vars. |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
Comments suppressed due to low confidence (5)
run/test/specs/linked_group_leave.spec.ts:1
ConversationItemis being constructed withdevice2while the wait is executed ondevice4. This will build a locator for the wrong platform/session context and can make the wait fail or behave nondeterministically. Construct the locator withdevice4instead.
run/test/utils/device_registry.ts:1
open_app.tsimportsregisterDevicesForTestfromdevice_registry.ts, whiledevice_registry.tsimportsSupportedPlatformsTypefromopen_app.ts, creating a circular module dependency at runtime. Make this a type-only import (import type) or moveSupportedPlatformsTypeinto a small shared types module (e.g.platform.ts) to break the cycle cleanly.
run/test/utils/utilities.ts:1
- This wrapper only catches synchronous matcher failures. Many Playwright matchers (e.g.
expect(locator).toBeVisible()) return a Promise that rejects asynchronously, which will bypass thistry/catchand still leak full diffs into Allure. Consider detecting promise returns and attaching a.catch(...)to rethrownew Error(message)(and also wrapping other chain properties likeresolves/rejectssimilarly tonot).
run/test/utils/community.ts:1
- The thrown error includes a newline and indentation, which can make CI logs noisy and harder to grep. Consider collapsing to a single-line message (or explicitly format with
\nwithout extra whitespace) so it renders consistently in logs/reporting.
run/test/specs/recovery_banner.spec.ts:1
- Function name casing is inconsistent (
Notshow), which makes it harder to scan and search. Rename tobannerShouldNotShowto follow standard camelCase conventions.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
both platforms accept message request with button both platforms scroll up after hiding recovery password
- disable lfs lock verification on publish step (macOS keychain error -25308) - disable credential helper on fetch-allure-history clone (same root cause) - extend prune workflow to also delete report dirs older than 60 days - refactor prune step to use shared bash functions (is_older_than, apply_deletions) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…tion Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
feat: qr code scanning
This file contains hidden or 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
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.