Skip to content
This repository was archived by the owner on Jan 16, 2025. It is now read-only.

Bump ts and other libs #1699

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Bump ts and other libs #1699

wants to merge 1 commit into from

Conversation

JSReds
Copy link
Contributor

@JSReds JSReds commented Mar 21, 2023

Change-type: patch
See: https://balena.zulipchat.com/#narrow/stream/346007-balena-io.2FbalenaCloud/topic/TS.20bump

still to solve:

  • Error coming by the export of Terminal (index.ts)
  • commented test not passing

Connects-to: #
See:
Depends-on:
Change-type: major|minor|patch


Contributor checklist
  • I have regenerated jest snapshots for any affected components with npm run generate-snapshots
Reviewer Guidelines
  • When submitting a review, please pick:
    • 'Approve' if this change would be acceptable in the codebase (even if there are minor or cosmetic tweaks that could be improved).
    • 'Request Changes' if this change would not be acceptable in our codebase (e.g. bugs, changes that will make development harder in future, security/performance issues, etc).
    • 'Comment' if you don't feel you have enough information to decide either way (e.g. if you have major questions, or you don't understand the context of the change sufficiently to fully review yourself, but want to make a comment)

@JSReds JSReds self-assigned this Mar 21, 2023
@JSReds JSReds marked this pull request as draft March 21, 2023 16:26
@@ -1,18 +1,11 @@
import cloneDeep from 'lodash/cloneDeep';
import * as React from 'react';
import styled from 'styled-components';
import styled, { withTheme } from 'styled-components';
import { ITerminalOptions, Terminal as Xterm } from 'xterm';
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the initialization of Xterm seems to be the problem for the error:

    TypeError: Cannot read properties of undefined (reading 'indexOf')



      at Object.6114 (node_modules/xterm/lib/webpack:/xterm/src/common/Platform.ts:43:14)
      at exports (node_modules/xterm/lib/webpack:/xterm/webpack/bootstrap:19:68)
      at Object.9312 (node_modules/xterm/lib/webpack:/xterm/src/browser/services/SelectionService.ts:11:1)
      at exports (node_modules/xterm/lib/webpack:/xterm/webpack/bootstrap:19:68)
      at Object.3236 (node_modules/xterm/lib/webpack:/xterm/src/browser/Terminal.ts:33:1)
      at exports (node_modules/xterm/lib/webpack:/xterm/webpack/bootstrap:19:68)
      at node_modules/xterm/lib/webpack:/xterm/src/browser/public/Terminal.ts:11:1
      at node_modules/xterm/lib/xterm.js:1:275629
      at node_modules/xterm/lib/xterm.js:1:275635
      at node_modules/xterm/lib/webpack:/xterm/webpack/universalModuleDefinition:4:28
      at Object.<anonymous> (node_modules/xterm/lib/webpack:/xterm/webpack/universalModuleDefinition:11:1)
      at Object.<anonymous> (src/components/Terminal/index.tsx:3954:28)
      at Object.<anonymous> (src/index.ts:6670:18)

Comment on lines +170 to +171
"testPathIgnorePatterns": ["<rootDir>/src/components/Terminal", "<rootDir>/node_modules/xterm", "<rootDir>/node_modules/xterm/", "<rootDir>/node_modules/xterm/*", "node_modules/xterm", "node_modules/xterm/", "node_modules/xterm/*"],
"modulePathIgnorePatterns": ["<rootDir>/src/components/Terminal", "<rootDir>/node_modules/xterm", "<rootDir>/node_modules/xterm/", "<rootDir>/node_modules/xterm/*", "node_modules/xterm", "node_modules/xterm/", "node_modules/xterm/*"]
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove after test

@@ -96,6 +96,7 @@ export {
} from './components/Table';

export { Accordion, AccordionProps } from './components/Accordion';
/* istanbul ignore next */
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove

Change-type: patch
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant