Merge remote-tracking branch 'upstream/dev'

Author: NikolaRHristov

.github/CODEOWNERS

@@ -0,0 +1,3 @@
+# At least one of the code owners below will be required on each PR:
+
+* @markAtMicrosoft @smmatte @adclements @abmahdy @iennae @nasadigital @isadorasophia @sagarmanchanda

.github/PULL_REQUEST_TEMPLATE.md

@@ -0,0 +1,33 @@
+# Description:
+
+Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
+
+## Type of change
+
+Please delete options that are not relevant.
+
+- [ ] Bug fix (non-breaking change which fixes an issue)
+- [ ] New feature (non-breaking change which adds functionality)
+- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
+- [ ] This change requires a documentation update
+
+# Limitations:
+
+Please describe limitations of this PR
+
+# Testing:
+
+Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
+
+- [ ] Test A
+- [ ] Test B
+
+# Checklist:
+
+- [ ] My code follows the style guidelines of this project
+- [ ] My code has been formatted with `npm run format` and passes the checks in `npm run check`
+- [ ] I have performed a self-review of my own code
+- [ ] I have commented my code, particularly in hard-to-understand areas
+- [ ] I have made corresponding changes to the documentation
+- [ ] My changes generate no new warnings
+- [ ] Any dependent changes have been merged and published in downstream modules

.github/workflows/ci.yml

@@ -0,0 +1,52 @@
+# This workflow will do a clean install of node dependencies, python dependencies, build the source code and run tests
+
+name: CI
+
+on:
+  push:
+    branches: [ dev, staging ]
+  pull_request:
+    branches: [ dev, staging ]
+
+jobs:
+  build-and-test:
+
+    runs-on: ${{ matrix.os }}
+
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [macos-latest, windows-latest, ubuntu-18.04]
+        node-version: [10.x]
+
+    steps:
+    - uses: actions/checkout@v2
+    - name: Install Linux dependencies
+      if: matrix.os == 'ubuntu-18.04'
+      run: |
+        sudo apt-get update
+        sudo apt-get install g++-multilib
+        sudo apt-get install -y build-essential
+        sudo apt-get install libudev-dev
+    - name: Use Node.js ${{ matrix.node-version }} and install npm dependencies
+      uses: actions/setup-node@v2
+      with:
+        node-version: ${{ matrix.node-version }}
+    - run: npm install
+    - name: Setup Python environment
+      uses: actions/setup-python@v2
+      with:
+        python-version: 3.x
+    - run: |
+        python -m pip install --upgrade pip
+        pip install -r ./src/dev-requirements.txt
+    - name: Use npm to compile, format-check and test
+      uses: actions/setup-node@v1
+      with:
+        node-version: ${{ matrix.node-version }}
+    - run: npm run compile
+    - run: npm run check
+    - name: Run tests
+      uses: GabrielBB/xvfb-action@v1.0
+      with: 
+        run: npm run test

.prettierrc.yaml

@@ -0,0 +1,5 @@
+trailingComma: "es5"
+tabWidth: 4
+semi: true
+endOfLine: auto
+printWidth: 80

.vscode/extensions.json

@@ -0,0 +1,7 @@
+{
+	// See http://go.microsoft.com/fwlink/?LinkId=827846
+	// for the documentation about the extensions.json format
+	"recommendations": [
+		"ms-vscode.vscode-typescript-tslint-plugin"
+	]
+}

.vscode/launch.json

@@ -0,0 +1,30 @@
+// A launch configuration that compiles the extension and then opens it inside a new window
+// Use IntelliSense to learn about possible attributes.
+// Hover to view descriptions of existing attributes.
+// For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
+{
+  "version": "0.2.0",
+  "configurations": [
+    {
+      "name": "Run Extension",
+      "type": "extensionHost",
+      "request": "launch",
+      "runtimeExecutable": "${execPath}",
+      "args": ["--extensionDevelopmentPath=${workspaceFolder}"],
+      "outFiles": ["${workspaceFolder}/out/**/*.js"],
+      "preLaunchTask": "npm: watch"
+    },
+    {
+      "name": "Extension Tests",
+      "type": "extensionHost",
+      "request": "launch",
+      "runtimeExecutable": "${execPath}",
+      "args": [
+        "--extensionDevelopmentPath=${workspaceFolder}",
+        "--extensionTestsPath=${workspaceFolder}/out/test/suite/index"
+      ],
+      "outFiles": ["${workspaceFolder}/out/test/**/*.js"],
+      "preLaunchTask": "npm: watch"
+    }
+  ]
+}

.vscode/settings.json

@@ -0,0 +1,10 @@
+// Place your settings in this file to overwrite default and user settings.
+{
+    "files.exclude": {
+        "out": false // set this to true to hide the "out" folder with the compiled JS files
+    },
+    "search.exclude": {
+        "out": true // set this to false to include "out" folder in search results
+    }
+    // Turn off tsc task auto detection since we have the necessary tasks as npm scripts
+}

.vscode/tasks.json

@@ -0,0 +1,25 @@
+// See https://go.microsoft.com/fwlink/?LinkId=733558
+// for the documentation about the tasks.json format
+{
+  "version": "2.0.0",
+  "tasks": [
+    {
+      "type": "npm",
+      "script": "watch",
+      "problemMatcher": "$tsc-watch",
+      "isBackground": true,
+      "presentation": {
+        "reveal": "never"
+      },
+      "group": {
+        "kind": "build",
+        "isDefault": true
+      }
+    },
+    {
+      "type": "gulp",
+      "task": "add-locales",
+      "problemMatcher": []
+    }
+  ]
+}

CHANGELOG.md

@@ -1,11 +1,9 @@
 # Change Log
 
-All notable changes to the "embedded-python" extension will be documented in
-this file.
+All notable changes to the "embedded-python" extension will be documented in this file.
 
-Check [Keep a Changelog](http://keepachangelog.com/) for recommendations on how
-to structure this file.
+Check [Keep a Changelog](http://keepachangelog.com/) for recommendations on how to structure this file.
 
 ## [Unreleased]
 
--   Initial release
+- Initial release

CODING_STANDARDS.md

@@ -2,31 +2,24 @@
 
 ## Coding guidelines for TypeScript
 
--   The following standards are inspired from
-    [Coding guidelines for TypeScript](https://github.com/Microsoft/TypeScript/wiki/Coding-guidelines)
-    (which you should follow when something is not specified in this document,
-    although any pre-existing practices in a file being edited trump either
-    style guide).
+-   The following standards are inspired from [Coding guidelines for TypeScript](https://github.com/Microsoft/TypeScript/wiki/Coding-guidelines) (which you should follow when something is not specified in this document, although any pre-existing practices in a file being edited trump either style guide).
 
 ### Names
 
 -   Use `PascalCase` for type names.
--   Use `I` as a prefix for interface names only when an interface is
-    implemented by a class.
+-   Use `I` as a prefix for interface names only when an interface is implemented by a class.
 -   Use `PascalCase` for enum values.
 -   Use `camelCase` for function names.
 -   Use `camelCase` for property names and local variables.
--   Do not use `_` as a prefix for private properties (unless used as backing
-    properties).
+-   Do not use `_` as a prefix for private properties (unless used as backing properties).
 -   Use whole words in names when possible.
 
 ### Types
 
--   Do not export types/functions unless you need to share it across multiple
-    components.
+-   Do not export types/functions unless you need to share it across multiple components.
 -   Do not introduce new types/values to the global namespace.
--   Shared types should be defined in `types.ts`. Within a file, type
-    definitions should come first.
+-   Shared types should be defined in `types.ts`.
+    Within a file, type definitions should come first.
 
 ### null and undefined
 
@@ -44,21 +37,16 @@ Use single quotes for strings.
 ### Imports
 
 -   Use ES6 module imports.
--   Do not use bare `import *`; all imports should either explicitly pull in an
-    object or import an entire module, otherwise you're implicitly polluting the
-    global namespace and making it difficult to figure out from code examination
-    where a name originates from.
+-   Do not use bare `import *`; all imports should either explicitly pull in an object or import an entire module, otherwise you're implicitly polluting the global namespace and making it difficult to figure out from code examination where a name originates from.
 
 ### Style
 
 -   Use `prettier` to format `TypeScript` and `JavaScript` code.
 -   Use arrow functions over anonymous function expressions.
--   Always surround loop and conditional bodies with curly braces. Statements on
-    the same line are allowed to omit braces.
+-   Always surround loop and conditional bodies with curly braces. Statements on the same line are allowed to omit braces.
 -   Open curly braces always go on the same line as whatever necessitates them.
 -   Parenthesized constructs should have no surrounding whitespace.
--   A single space follows commas, colons, and semicolons in those constructs.
-    For example:
+-   A single space follows commas, colons, and semicolons in those constructs. For example:
 
     -   `for (var i = 0, n = str.length; i < 10; i++) { }`
     -   `if (x < 10) { }`
@@ -70,5 +58,4 @@ Use single quotes for strings.
 
 ## Coding Standards for Python
 
-Please follow the
-[Google Python Style Guide](https://google.github.io/styleguide/pyguide.html).
+Please follow the [Google Python Style Guide](https://google.github.io/styleguide/pyguide.html).