Merge pull request #2843 from input-output-hk/chore/ddw-773-migrate-to-typecript

Chore/ddw 773 migrate to typecript

Author: DominikGuzei

.babelrc

@@ -2,7 +2,7 @@
   "presets": [
     "@babel/preset-env",
     "@babel/preset-react",
-    "@babel/preset-flow"
+    "@babel/preset-typescript"
   ],
   "plugins": [
     ["@babel/plugin-proposal-decorators", { "legacy": true }],

.eslintrc

@@ -1,17 +1,17 @@
 {
-  "parser": "@babel/eslint-parser",
+  "parser": "@typescript-eslint/parser",
   "parserOptions": {
     "ecmaFeatures": {
       "legacyDecorators": true
     }
   },
   "extends": [
     "airbnb",
-    "plugin:flowtype/recommended",
+    "plugin:@typescript-eslint/recommended",
     "plugin:react/recommended",
     "prettier",
     "prettier/babel",
-    "prettier/flowtype",
+    "prettier/@typescript-eslint",
     "prettier/react",
     "plugin:jest/recommended"
   ],
@@ -24,7 +24,7 @@
   "rules": {
     "class-methods-use-this": 0,
     "consistent-return": "warn",
-    "flowtype/generic-spacing": 0,
+    "@typescript-eslint/generic-spacing": 0,
     "jsx-a11y/alt-text": 0,
     "lines-between-class-members": 0,
     "global-require": 0,
@@ -71,17 +71,22 @@
     "react/prefer-stateless-function": 0,
     "react/prop-types": "warn",
     "react/require-default-props": 0,
+    "react/jsx-first-prop-new-line": [1, "multiline-multiprop"],
+    "@typescript-eslint/ban-ts-comment": 1,
+    "@typescript-eslint/no-empty-function": 1,
+    "@typescript-eslint/ban-types": 1,
+    "import/no-unresolved": 1,
+    "@typescript-eslint/no-var-requires": 1,
+    "camelcase": 1,
+    "no-empty": 1,
+    "@typescript-eslint/no-explicit-any": 1,
+    "no-shadow": 1,
+    "react/no-did-update-set-state": 1,
     "react/sort-comp": 0,
     "react/state-in-constructor": 0,
     "react/static-property-placement": 0
   },
-  "plugins": [
-    "flowtype",
-    "import",
-    "promise",
-    "react",
-    "jest"
-  ],
+  "plugins": ["@typescript-eslint", "import", "promise", "react", "jest"],
   "globals": {
     "API": true,
     "API_VERSION": true,
@@ -91,6 +96,8 @@
     "Process": true // TODO: remove after fix
   },
   "settings": {
-    "import/resolver": "webpack"
+    "import/resolver": {
+      "node": { "extensions": [".js", ".jsx", ".ts", ".tsx"] }
+    }
   }
 }

.flowconfig

@@ -20,7 +20,7 @@ or animated GIFs of important UI changes in both English and Japanese.
 Do not use shadow or any effects. On macOS this can be accomplished the following way:
 1. Use the Command+Shift+4 keyboard shortcut.
 2. Press the Spacebar.
-3. Hold the Option button and click the window you want to capture. 
+3. Hold the Option button and click the window you want to capture.
 -->
 
 ## Testing Checklist
@@ -29,7 +29,6 @@ Do not use shadow or any effects. On macOS this can be accomplished the followin
 Open a thread on #daedalus-qa on Slack, mention `@daedalusqa` and `@daedalusteam`, link the thread below
 -->
 
-
 - [Slack QA thread](https://input-output-rnd.slack.com/messages/GGKFXSKC6)
 - [ ] Test
 
@@ -38,7 +37,8 @@ Open a thread on #daedalus-qa on Slack, mention `@daedalusqa` and `@daedalusteam
 ## Review Checklist
 
 ### Basics
-- [ ] PR assigned to the PR author(s) 
+
+- [ ] PR assigned to the PR author(s)
 - [ ] `input-output-hk/daedalus-dev` and `input-output-hk/daedalus-qa` assigned as PR reviewers
 - [ ] If there are UI changes, Alexander Rukin assigned as an additional reviewer
 - [ ] All visual regression testing has been reviewed (Go to Github Actions tab -> Select Chromatic workflow -> Run on your working branch)
@@ -47,25 +47,28 @@ Open a thread on #daedalus-qa on Slack, mention `@daedalusqa` and `@daedalusteam
 - [ ] PR is updated to the most recent version of the target branch (and there are no conflicts)
 - [ ] PR has a good description that summarizes all changes
 - [ ] PR contains screenshots (in case of UI changes)
-- [ ] CHANGELOG entry has been added to the top of the appropriate section (*Features*, *Fixes*, *Chores*) and is linked to the correct PR on GitHub
+- [ ] CHANGELOG entry has been added to the top of the appropriate section (_Features_, _Fixes_, _Chores_) and is linked to the correct PR on GitHub
 - [ ] There are no missing translations (running `yarn manage:translations` produces no changes)
 - [ ] Text changes are proofread and approved (Jane Wild / Amy Reeve)
 - [ ] Japanese text changes are proofread and approved (Junko Oda)
 - [ ] Storybook works and no stories are broken (`yarn storybook`)
 - [ ] In case of dependency changes `yarn.lock` file is updated
 
 ### Code Quality
+
 - [ ] Important parts of the code are properly commented and documented
-- [ ] Code is properly typed with flow
+- [ ] Code is properly typed with typescript types
 - [ ] React components are split-up enough to avoid unnecessary re-renderings
 - [ ] Any code that only works in main process is neatly separated from components
 
 ### Testing
+
 - [ ] New feature/change is covered by acceptance tests
 - [ ] New feature/change is manually tested and approved by QA team
 - [ ] All existing acceptance tests are still up-to-date
 - [ ] New feature/change is covered by Daedalus Testing scenario
 - [ ] All existing Daedalus Testing scenarios are still up-to-date
 
 ### After Review
+
 - [ ] Update Slack QA thread by marking it with a green checkmark

.github/pull_request_template.md

@@ -10,6 +10,6 @@ jobs:
       - name: Setup Node.js
         uses: actions/setup-node@v2
         with:
-          node-version: "14"
+          node-version: '14'
       - run: yarn install --frozen-lockfile
-      - run: yarn prettier:check && yarn lint && yarn stylelint
+      - run: yarn prettier:check && yarn compile && yarn lint && yarn stylelint

.github/workflows/linter.yml

@@ -11,6 +11,8 @@
 
 # But not these file types
 !*.js
+!*.ts
+!*.tsx
 !*.scss
 !*.json
 

.prettierignore

@@ -11,6 +11,7 @@
 
 ### Chores
 
+- Migrated codebase from javascript to typescript ([PR 2843](https://github.com/input-output-hk/daedalus/pull/2843))
 - Updated the list of team members ([PR 2805](https://github.com/input-output-hk/daedalus/pull/2805))
 
 ## 4.8.0-FC1

CHANGELOG.md

@@ -0,0 +1,45 @@
+declare module '*.svg' {
+  const content: any;
+  export default content;
+}
+
+declare module '*.scss' {
+  const content: any;
+  export default content;
+}
+
+type Daedalus = {
+  actions: ActionsMap;
+  api: Api;
+  environment: Object;
+  reset: Function;
+  stores: StoresMap;
+  translations: Object;
+  utils: {
+    crypto: {
+      generateMnemonic: Function;
+    };
+  };
+};
+
+export type $ElementType<
+  T extends { [P in K & any]: any },
+  K extends keyof T | number
+> = T[K];
+
+export type EnumMap<
+  K extends string,
+  V,
+  O extends Record<string, any> = any
+> = O & Record<K, V & $ElementType<O, K>>;
+
+declare global {
+  namespace NodeJS {
+    interface ProcessEnv {
+      WALLET_COUNT: number;
+    }
+  }
+  var daedalus: Daedalus;
+}
+
+export {};

declaration.d.ts

@@ -319,7 +319,7 @@ let
     electron = pkgs.callPackage ./installers/nix/electron.nix {};
 
     tests = {
-      runFlow = self.callPackage ./tests/flow.nix {};
+      runTsc = self.callPackage ./tests/tsc.nix {};
       runLint = self.callPackage ./tests/lint.nix {};
       runShellcheck = self.callPackage ./tests/shellcheck.nix { src = ./.;};
     };