Merge 'master' branch into 'io-fix-build-to-yaml'

.devcontainer/Dockerfile

@@ -1,9 +1,9 @@
 #----------------------------------------------------------------------------------------------
-# Copyright (c) 2021 Bentley Systems, Incorporated. All rights reserved.
+# Copyright (c) 2022 Bentley Systems, Incorporated. All rights reserved.
 # Licensed under the MIT License. See LICENSE.md in the project root for license terms.
 #----------------------------------------------------------------------------------------------
 
-FROM debian:10
+FROM docker.io/library/debian:11
 
 RUN \
   apt-get update \
@@ -15,7 +15,8 @@ RUN \
   libpango-1.0-0 libpangocairo-1.0-0 libstdc++6 libx11-6 libx11-xcb1 \
   libxcb1 libxcomposite1 libxcursor1 libxdamage1 libxext6 libxfixes3 \
   libxi6 libxrandr2 libxrender1 libxss1 libxtst6 ca-certificates \
-  fonts-liberation libappindicator1 libnss3 lsb-release xdg-utils wget \
+  fonts-liberation libayatana-appindicator1 libnss3 libsecret-1-0 \
+  lsb-release xdg-utils wget \
   && apt-get install -y git nodejs locales zsh procps \
   && npm install -g @microsoft/rush \
   && localedef -i en_US -c -f UTF-8 -A /usr/share/locale/locale.alias en_US.UTF-8

.github/CODEOWNERS

@@ -19,7 +19,6 @@
 rush.json       @iTwin/itwinjs-core-admins
 
 /clients                        @calebmshafer @ramanujam-raman
-/clients/imodelhub              @amikoliunas @Animagne
 
 # The syntax used below tries to cover both files and folders in the common folder that operate per=package
 # Due to syntax requirements, both can't be covered in a single line
@@ -40,8 +39,6 @@ rush.json       @iTwin/itwinjs-core-admins
 # Covers most client libraries, all others with more specific reviewers should be after these two lines
 /common/**/*-client.*                         @calebmshafer @ramanujam-raman
 /common/**/*-client/                          @calebmshafer @ramanujam-raman
-/common/**/imodelhub-client.*                 @amikoliunas @Animagne
-/common/**/imodelhub-client/                  @amikoliunas @Animagne
 
 /common/**/*-extension/                       @bsteinbk @calebmshafer
 
@@ -56,8 +53,8 @@ rush.json       @iTwin/itwinjs-core-admins
 /common/**/express-server/                    @calebmshafer @wgoehrig
 /common/**/frontend-devtools.*                @iTwin/itwinjs-core-display @Ellord207
 /common/**/frontend-devtools/                 @iTwin/itwinjs-core-display @Ellord207
-/common/**/core-geometry.*                    @bbastings @EarlinLutz @mgooding @RBBentley @dassaf4
-/common/**/core-geometry/                     @bbastings @EarlinLutz @mgooding @RBBentley @dassaf4
+/common/**/core-geometry.*                    @bbastings @mgooding @dassaf4
+/common/**/core-geometry/                     @bbastings @mgooding @dassaf4
 /common/**/hypermodeling-frontend.*           @iTwin/itwinjs-core-display @bbastings
 /common/**/hypermodeling-frontend/            @iTwin/itwinjs-core-display @bbastings
 /common/**/core-backend.*                     @iTwin/itwinjs-core-admins
@@ -80,7 +77,7 @@ rush.json       @iTwin/itwinjs-core-admins
 /common/**/linear-referencing-backend/        @calebmshafer @diegoalexdiaz
 /common/**/linear-referencing-common.*        @calebmshafer @diegoalexdiaz
 /common/**/linear-referencing-common/         @calebmshafer @diegoalexdiaz
-/common/**/map-layers/                        @bsteinbk @mdastous-bentley
+/common/**/map-layers/                        @mdastous-bentley @MarcBedard8
 /common/**/oidc-signin-tool/                  @calebmshafer @wgoehrig
 /common/**/core-orbitgt.*                     @iTwin/itwinjs-core-display
 /common/**/core-orbitgt/                      @iTwin/itwinjs-core-display
@@ -108,10 +105,11 @@ rush.json       @iTwin/itwinjs-core-admins
 /core/ecschema-*                @calebmshafer @ColinKerr @wgoehrig
 /core/electron                  @wgoehrig @ramanujam-raman
 /core/express-server            @calebmshafer @wgoehrig @ramanujam-raman
+/core/extension                 @calebmshafer @wgoehrig @aruniverse @kckst8
 /core/frontend                  @kabentley @bbastings @iTwin/itwinjs-core-display
-/core/frontend/src/extension    @calebmshafer @wgoehrig
+/core/frontend/src/extension    @calebmshafer @wgoehrig @aruniverse @kckst8
 /core/frontend-devtools         @iTwin/itwinjs-core-display @Ellord207
-/core/geometry                  @bbastings @EarlinLutz @mgooding @RBBentley @dassaf4
+/core/geometry                  @bbastings @mgooding @dassaf4
 /core/hypermodeling             @bbastings @iTwin/itwinjs-core-display
 /core/i18n                      @wgoehrig @calebmshafer
 /core/markup                    @kabentley @pmconne @bbastings
@@ -137,11 +135,11 @@ rush.json       @iTwin/itwinjs-core-admins
 /example-code                         @iTwin/itwinjs-core-admins
 
 /extensions                           @bsteinbk @calebmshafer
+/extensions/test-extension            @calebmshafer @wgoehrig @aruniverse @kckst8
 /extensions/map-layers                @bsteinbk @mdastous-bentley
 
 /full-stack-tests/core                @iTwin/itwinjs-core-admins
 /full-stack-tests/ecschema-rpc-interface @calebmshafer @ColinKerr @wgoehrig
-/full-stack-tests/imodelhub-client    @iTwin/itwinjs-core-admins
 /full-stack-tests/presentation  @iTwin/itwinjs-core-presentation
 /full-stack-tests/rpc           @calebmshafer @wgoehrig @swbsi
 /full-stack-tests/rpc-interface @calebmshafer @wgoehrig
@@ -153,21 +151,21 @@ rush.json       @iTwin/itwinjs-core-admins
 /test-apps/display-test-app                 @bbastings @kabentley @iTwin/itwinjs-core-display
 /test-apps/export-gltf                      @mgooding @MichaelBelousov
 /test-apps/export-obj                       @mgooding @MichaelBelousov
-/test-apps/imjs-importer                    @EarlinLutz @dassaf4
+/test-apps/imjs-importer                    @bbastings @dassaf4
 /test-apps/imodel-from-geojson              @mgooding @pmconne
 /test-apps/imodel-from-orbitgt              @iTwin/itwinjs-core-display
 /test-apps/imodel-from-reality-model        @iTwin/itwinjs-core-display
 /test-apps/imodel-transformer               @MichaelBelousov @ColinKerr
 /test-apps/ninezone-test-app                @iTwin/itwinjs-core-ui
 /test-apps/presentation-test-app            @iTwin/itwinjs-core-presentation
-/test-apps/synchro-schedule-importer        @RBBentley @pmconne
+/test-apps/synchro-schedule-importer        @iTwin/itwinjs-core-display
 /test-apps/ui-test-app                      @iTwin/itwinjs-core-ui
 /test-apps/ui-items-providers-test          @iTwin/itwinjs-core-ui
 
 /tools/backend-webpack @calebmshafer @wgoehrig
 /tools/build @calebmshafer @wgoehrig
 /tools/certa @calebmshafer @wgoehrig
-/tools/webpack-core @calebmshafer @wgoehrig
+/tools/webpack-core @calebmshafer @wgoehrig @aruniverse
 /tools/ecschema2ts @calebmshafer @ColinKerr @wgoehrig
 /tools/eslint-plugin @calebmshafer @paulius-valiunas
 /tools/internal @calebmshafer @wgoehrig

.github/workflows/extract-api.yaml

@@ -1,4 +1,4 @@
-# iModel.js Extract API Build
+# iTwin.js Extract API Build
 
 name: Extract API
 

.vscode/launch.json

@@ -358,62 +358,6 @@
         "${workspaceFolder}/{core,clients}/*/lib/**/*.js"
       ]
     },
-    {
-      "name": "iModelHub Client Tests (iModelBank)",
-      "presentation": {
-        "group": "3_ClientsTests",
-        "order": 3
-      },
-      "cwd": "${workspaceFolder}/full-stack-tests/imodelhub-client",
-      "type": "pwa-node",
-      "request": "launch",
-      "runtimeExecutable": "npm",
-      "runtimeArgs": [
-        "run",
-        "test:imodel-bank"
-      ],
-      "outFiles": [
-        "${workspaceFolder}/{core,clients}/*/lib/**/*.js",
-        "${workspaceFolder}/full-stack-tests/imodelhub-client/lib/**/*.js"
-      ]
-    },
-    {
-      "name": "iModelHub Client Tests (Integration)",
-      "presentation": {
-        "group": "3_ClientsTests",
-        "order": 3
-      },
-      "cwd": "${workspaceFolder}/full-stack-tests/imodelhub-client",
-      "type": "pwa-node",
-      "request": "launch",
-      "runtimeExecutable": "npm",
-      "runtimeArgs": [
-        "run",
-        "test:integration"
-      ],
-      "outFiles": [
-        "${workspaceFolder}/{core,clients}/*/lib/**/*.js",
-        "${workspaceFolder}/full-stack-tests/imodelhub-client/lib/**/*.js"
-      ]
-    },
-    {
-      "name": "iModelHub Client Tests (Offline)",
-      "presentation": {
-        "group": "3_ClientsTests",
-        "order": 3
-      },
-      "cwd": "${workspaceFolder}/full-stack-tests/imodelhub-client",
-      "type": "pwa-node",
-      "request": "launch",
-      "runtimeExecutable": "npm",
-      "runtimeArgs": [
-        "test"
-      ],
-      "outFiles": [
-        "${workspaceFolder}/{core,clients}/*/lib/**/*.js",
-        "${workspaceFolder}/full-stack-tests/imodelhub-client/lib/**/*.js"
-      ]
-    },
     {
       "name": "iTwin Client Tests (Offline)",
       "presentation": {
@@ -443,7 +387,7 @@
       "program": "${workspaceFolder}/ui/appui-abstract/node_modules/mocha/bin/_mocha",
       "args": [
         "--config",
-        "../.mocharc.json",
+        "../.mocharc-noreact.json",
         "--no-timeouts",
         "lib/cjs/test/**/*.test.js"
       ],
@@ -668,14 +612,10 @@
       "request": "launch",
       "program": "${workspaceFolder}/full-stack-tests/presentation/node_modules/mocha/bin/_mocha",
       "args": [
-        "-r",
-        "jsdom-global/register",
-        "-r",
-        "ignore-styles",
-        "lib/**/*.js"
+        "--no-config"
       ],
       "env": {
-        "NODE_ENV": "development",
+        "NODE_ENV": "development"
       },
       "outFiles": [
         "${workspaceFolder}/{core,clients,ui,presentation}/*/lib/**/*.js",
@@ -824,6 +764,24 @@
         "${workspaceFolder}/test-apps/imodel-from-geojson/lib/**/*.js"
       ]
     },
+    {
+      "name": "workspace-editor",
+      "presentation": {
+        "group": "6_Misc"
+      },
+      "cwd": "${workspaceFolder}/utils/workspace-editor",
+      "type": "pwa-node",
+      "request": "launch",
+      "program": "${workspaceFolder}/utils/workspace-editor/lib/WorkspaceEditor.js",
+      "args": [
+        "download",
+        "usa-nsrs2011",
+        "-l=d:/temp/abc.d"
+      ],
+      "outFiles": [
+        "${workspaceFolder}/utils/workspace-editor/lib/**/*.js"
+      ]
+    },
     {
       "name": "imodel-transformer (app)",
       "presentation": {
@@ -1239,50 +1197,6 @@
         "[FRONTEND] Full Stack Core Integration Tests"
       ]
     },
-    { /* PARTIAL */
-      "name": "[FRONTEND] Full Stack IModelClient Integration Tests",
-      "presentation": {
-        "hidden": true
-      },
-      "type": "pwa-chrome",
-      "request": "attach",
-      "port": 9223, // NB: This must match ports.frontendDebugging in full-stack-tests/core/certa.json
-      "timeout": 200000,
-      "outFiles": [
-        "${workspaceFolder}/full-stack-tests/imodelhub-client/lib/**/*.js",
-        "${workspaceFolder}/{core,clients}/*/lib/**/*.js",
-        "${workspaceFolder}/tools/certa/lib/**/*.js"
-      ],
-      "cascadeTerminateToConfigurations": [
-        "[BACKEND] Full Stack IModelClient Integration Tests"
-      ]
-    },
-    { /* PARTIAL */
-      "name": "[BACKEND] Full Stack IModelClient Integration Tests",
-      "presentation": {
-        "hidden": true
-      },
-      "cwd": "${workspaceFolder}/full-stack-tests/native-app/",
-      "type": "pwa-node",
-      "request": "launch",
-      "runtimeExecutable": "npm",
-      "runtimeArgs": [
-        "run",
-        "test:integration:electron",
-        "--",
-        "--debug"
-      ],
-      "outFiles": [
-        "${workspaceFolder}/full-stack-tests/imodelhub-client/lib/**/*.js",
-        "${workspaceFolder}/{core,clients}/*/lib/**/*.js",
-        "${workspaceFolder}/tools/certa/lib/**/*.js"
-      ],
-      "outputCapture": "std",
-      "attachSimplePort": 5858, // NB: This must match ports.debugging in full-stack-tests/native-app/certa.json
-      "cascadeTerminateToConfigurations": [
-        "[FRONTEND] Full Stack IModelClient Integration Tests"
-      ]
-    },
     { /* PARTIAL */
       "name": "[FRONTEND] Full Stack Core Integration Tests",
       "presentation": {
@@ -1629,4 +1543,4 @@
       ]
     }
   ]
-}
+}

.vscode/settings.json

@@ -115,4 +115,4 @@
     "editor.action.fixAll": true, // Run the default formatter before eslint, it handles whitespace better
     "source.fixAll.eslint": true // then run ESLint auto-fix
   },
-}
+}

CONTRIBUTING.md

@@ -70,12 +70,9 @@ There are just a few guidelines you need to follow.
 
 ### Contributor License Agreement (CLA)
 
-You must sign a [Contribution License Agreement with Bentley](Bentley-CLA.pdf) before your contributions will be accepted.
-This a one-time requirement for Bentley projects in GitHub.
+A [Contribution License Agreement with Bentley](https://gist.github.com/imodeljs-admin/9a071844d3a8d420092b5cf360e978ca) must be signed before your contributions will be accepted. Upon opening a pull request, you will be prompted to use [cla-assistant](https://cla-assistant.io/) for a one-time acceptance applicable for all Bentley projects.
 You can read more about [Contributor License Agreements](https://en.wikipedia.org/wiki/Contributor_License_Agreement) on Wikipedia.
 
-> Note: a CLA is not required if the change is trivial (such as fixing a spelling error or a typo).
-
 ### Pull Requests
 
 All submissions go through a review process.

LICENSE.md

@@ -1,6 +1,6 @@
 # MIT License
 
-Copyright © 2017-2021 Bentley Systems, Incorporated. All rights reserved.
+Copyright © 2017-2022 Bentley Systems, Incorporated. All rights reserved.
 
 Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the "Software"), to deal in the Software without restriction, including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, subject to the following conditions:
 

Steering.md

@@ -0,0 +1,29 @@
+# iTwin.js Steering Committee Charter 
+
+The goal of the iTwin.js Steering Committee is to ensure that iTwin.js meets the needs of its users. The Steering Committee represents the voice of the user and works with iTwin.js maintainers to help guide development efforts towards solving problems for the iTwin.js community. 
+
+## Responsibilities 
+
+The Steering Committee’s responsibilities are to: 
+
+1. Provide feedback to the development team about the functionality, accessibility, and usability of iTwin.js. 
+2. Assist iTwin.js development team in prioritizing upcoming roadmap items and planned work. 
+3. Represent the "voice of the iTwin.js user" to the broader community. 
+
+## Membership 
+
+The Steering Committee memberships are not time-limited nor is there a maximum size limit.  The size of the committee is expected to vary to ensure adequate coverage in area expertise and diversity of types of users.  The Steering Committee must have a minimum of four members, and no more than one-fourth of the members may be affiliated with the same employer.    
+
+To be eligible for membership in the Steering Committee, you must: 
+
+1. Be actively developing an application or solution using iTwin.js 
+2. Be willing and able to attend regularly scheduled Steering Committee meetings 
+
+Candidates for membership will be nominated by current Steering Committee members or by iTwin.js maintainers.  The addition of new members will be conducted by a standard motion and vote.  A member may be removed from the Steering Committee by voluntary resignation, by a standard committee motion and vote. 
+
+The Steering Committee will elect amongst its members a Chairperson to work with the iTwin.js maintainers to build an agenda for the meetings.  The committee shall hold annual elections to select a Chairperson for a term of one year.  There are no limits on the number of terms a Chairperson may serve.   
+
+## Meetings 
+
+Meetings will happen periodically not more than once a quarter. Recordings and minutes will be posted publicly. 
+Subcommittees can be formed and meet more frequently or use chat rooms to discuss topics in more detail if needed.  The findings of the subcommittees will be shared in the Steering Committee regular meetings.