Merge to master

.github/CODEOWNERS

@@ -0,0 +1,108 @@
+# This is a comment.
+# Each line is a file pattern followed by one or more owners.
+
+# These owners will be the default owners for everything in the repo.
+# Unless a later match takes precedence, @global-owner1 and @global-owner2
+# will be requested for review when someone opens a pull request.
+# *   @global-owner1 @global-owner2
+
+# Order is important; the last matching pattern takes the most precedence.
+# When someone opens a pull request that only modifies JS files, only @js-owner
+# and not the global owner(s) will be requested for a review.
+# *.js@js-owner
+
+# You can also use email addresses if you prefer. They'll be used to look up
+# users just like we do for commit author emails.
+# docs/  docs@example.com
+
+#### Core build stuff
+scripts/ @dzearing
+
+#### Apps
+# component-demo/
+# fabric-website/
+# ssr-tests/
+# test-bundle-button/ @dzearing
+# todo-app/
+
+#### Packages
+# packages/example-app-base/
+packages/file-type-icons/  @KatherineThayerMicrosoft @jahnp @bigbadcapers
+# packages/icons/
+# packages/jest-serializer-merge-styles/
+# packages/merge-styles/
+# packages/office-ui-fabric-react-tslint/
+packages/styling/ @dzearing
+packages/styling/src/interfaces/ @phkuo
+packages/styling/src/styles/ @phkuo
+# packages/utilities/
+packages/utilities/positioning/ @joschect
+
+### Fabric
+# common/
+common/_semanticSlots.scss  @phkuo
+common/_themeOverrides.scss @phkuo
+common/_common.scss @phkuo
+
+### Utilities
+dateMath/     @lorejoh12 @sathi23
+dateValues/   @lorejoh12 @sathi23
+
+## Components
+# ActivityItem/
+# Breadcrumb/
+# Button/
+Calendar/   @lorejoh12 @sathi23
+Callout/  @joschect
+# Check/
+# Checkbox/
+# ChoiceGroup/
+# ColorPicker/
+# ComboBox/
+CommandBar/   @micahgodbolt
+ContextualMenu/  @joschect
+DatePicker/   @lorejoh12 @sathi23
+# DetailsList/
+# Dialog/
+DocumentCard/ @yiminwu @mikewheaton
+# Dropdown/
+# Fabric/
+# FacePile/
+# FocusTrapZone/
+# FocusZone/
+GroupedList/  @aditima
+HoverCard/   @atneik @Jahnp
+# Icon/
+# Image/
+# Label/
+Layer/  @ThomasMichon
+# Link/
+List/  @cschleiden
+# MarqueeSelection/
+# MessageBar/
+# Modal/
+# Nav/
+OverflowSet/  @micahgodbolt
+# Overlay/
+# Panel/
+# Persona/
+Persona/PersonaCoin.tsx @mtennoe @jakob101
+Persona/PersonaConsts.tsx @mtennoe @jakob101
+pickers/  @joschect
+# Pivot/
+# Popup/
+# ProgressIndicator/
+Rating/  @cschleiden
+ResizeGroup/  @micahgodbolt
+# SearchBox/
+# Slider/
+# SpinButton/
+# Spinner/
+# SwatchColorPicker/
+TeachingBubble/   @micahgodbolt
+# TextField/
+# Theme/
+Toggle/  @phkuo
+Tooltip/  @micahgodbolt
+
+

.github/PULL_REQUEST_TEMPLATE.md

@@ -1,10 +1,7 @@
 #### Pull request checklist
 
 - [ ] Addresses an existing issue: Fixes #0000
-- [ ] Include a change request file if publishing <!-- see notes below -->
-- [ ] New feature, bugfix, or enhancement
-  - [ ] Includes tests
-- [ ] Documentation update
+- [ ] Include a change request file using `$ npm run change`
 
 #### Description of changes
 
@@ -13,19 +10,3 @@
 #### Focus areas to test
 
 (optional)
-
-<!--
-For change request files, you need to use the `rush` tool. You can globally install it:
-
-```
-npm i -g @microsoft/rush
-```
-
-To generate a file, you simply run:
-
-```
-rush change
-```
-
-This will ask you questions about your change and create a json file under the `common/changes` folder. The comments you include in the file will show up in the public changelog notes, so please follow the existing conventions. Commit this file and include it in your PR.
--->

.gitignore

@@ -42,6 +42,7 @@ node_modules
 temp
 lib
 lib-amd
+lib-es2015
 coverage
 dist
 npm-debug.log
@@ -104,11 +105,12 @@ package-deps.json
 **/office-ui-fabric-react/visualtests/results/*.png
 **/office-ui-fabric-react/failures/*.png
 
-common/npm-local
-common/npmx-link.json
-common/last-install.*
-common/npmx-recycler
+common/temp
 
-common/rush-link.json
-common/last-install.*
-common/rush-recycler
+# Gulp cache
+gulp-cache
+
+# NPM debug logs
+npm-debug.log.*
+
+package-lock.json

.npmrc

@@ -0,0 +1,2 @@
+registry=https://registry.npmjs.org/
+package-lock=false

.travis.yml

@@ -1,11 +1,10 @@
 language: node_js
 node_js:
-  - 'node'
-before_script:
-  - npm install -g gulp
+  - '8'
 script:
-  - npm run build
-cache:
-  directories:
-    - node_modules
-    - common/node_modules
+  - if [ $TRAVIS_BRANCH == "master" ] && [ $TRAVIS_PULL_REQUEST == "true" ]; then
+      npm run checkchange;
+    fi
+  - npm run buildfast
+  - npm run bundlesize
+  - npm run vrtest

.vscode/launch.json

@@ -2,13 +2,35 @@
   "version": "0.2.0",
   "configurations": [
     {
-      "name": "Launch",
+      "name": "Debug test",
       "type": "node",
       "request": "launch",
-      "program": "${workspaceRoot}/packages/office-ui-fabric-react/node_modules/gulp/bin/gulp.js",
+      "program": "${workspaceRoot}/scripts/debug-test.js",
+      "cwd": "${fileDirname}",
       "stopOnEntry": false,
-      "args": [],
+      "args": [
+        "-i"
+      ],
+      "runtimeExecutable": null,
+      "runtimeArgs": [
+        "--nolazy",
+        "--debug"
+      ],
+      "env": {
+        "NODE_ENV": "development"
+      },
+      "sourceMaps": true
+    },
+    {
+      "name": "Build fabric",
+      "type": "node",
+      "request": "launch",
+      "program": "${workspaceRoot}/scripts/build.js",
       "cwd": "${workspaceRoot}/packages/office-ui-fabric-react",
+      "stopOnEntry": false,
+      "args": [
+        "ts"
+      ],
       "runtimeExecutable": null,
       "runtimeArgs": [
         "--nolazy",
@@ -17,9 +39,50 @@
       "env": {
         "NODE_ENV": "development"
       },
-      "externalConsole": false,
-      "sourceMaps": false,
-      "outDir": null
+      "sourceMaps": true
+    },
+    {
+      "name": "Debug build Production",
+      "type": "node",
+      "request": "launch",
+      "program": "${workspaceRoot}/scripts/build.js",
+      "stopOnEntry": false,
+      "args": [
+        "webpack",
+        "--",
+        "--production"
+      ],
+      "cwd": "${workspaceRoot}/apps/fabric-website",
+      "runtimeExecutable": null,
+      "runtimeArgs": [
+        "--nolazy",
+        "--debug"
+      ],
+      "env": {
+        "NODE_ENV": "development"
+      }
+    },
+    {
+      "name": "Debug build Dogfood",
+      "type": "node",
+      "request": "launch",
+      "program": "${workspaceRoot}/scripts/build.js",
+      "stopOnEntry": false,
+      "args": [
+        "webpack",
+        "--",
+        "--production",
+        "--dogfood"
+      ],
+      "cwd": "${workspaceRoot}/apps/fabric-website",
+      "runtimeExecutable": null,
+      "runtimeArgs": [
+        "--nolazy",
+        "--debug"
+      ],
+      "env": {
+        "NODE_ENV": "development"
+      }
     },
     {
       "name": "Run ssr tests",
@@ -40,9 +103,28 @@
       "env": {
         "NODE_ENV": "development"
       },
-      "externalConsole": false,
-      "sourceMaps": false,
-      "outDir": null
+      "sourceMaps": false
     },
+    {
+      "name": "Debug add new component",
+      "type": "node",
+      "request": "launch",
+      "program": "${workspaceRoot}/scripts/create-component.js",
+      "cwd": "${workspaceRoot}",
+      "stopOnEntry": false,
+      "args": [
+        "--name",
+        "TestComponentName"
+      ],
+      "runtimeExecutable": null,
+      "runtimeArgs": [
+        "--nolazy",
+        "--debug"
+      ],
+      "env": {
+        "NODE_ENV": "development"
+      },
+      "sourceMaps": true
+    }
   ]
 }

.vscode/settings.json

@@ -12,27 +12,38 @@
   "files.exclude": {
     "**/.git": true,
     "**/.DS_Store": true,
-    "**/bower_components": true,
     "**/coverage": true,
     "**/*.scss.ts": true
   },
+  "files.watcherExclude": {
+    "**/.git": true,
+    "**/node_modules": true,
+    "**/lib": true,
+    "**/lib-amd": true,
+    "/**lib-es2015": true,
+    "**/temp": true,
+    "**/dist": true,
+    "**/*.scss.ts": true,
+    "**/coverage": true
+  },
   // Configure glob patterns for excluding files and folders in searches. Inherits all glob patterns from the file.exclude setting.
   "search.exclude": {
     "common": true,
     "**/node_modules": true,
     "**/lib": true,
     "**/lib-amd": true,
+    "**/lib-es2015": true,
     "**/dist": true,
     "**/*.scss.ts": true
   },
   // Controls after how many characters the editor will wrap to the next line. Setting this to 0 turns on viewport width wrapping
   "editor.wordWrap": "wordWrapColumn",
   "editor.wordWrapColumn": 140,
   "tslint.enable": true,
-  "tslint.rulesDirectory": "./common/node_modules/tslint-microsoft-contrib",
+  "tslint.rulesDirectory": "./common/temp/node_modules/tslint-microsoft-contrib",
   // Defines space handling after opening and before closing JSX expression braces. Requires TypeScript >= 2.0.6.
   "typescript.format.insertSpaceAfterOpeningAndBeforeClosingJsxExpressionBraces": true,
   "editor.formatOnSave": true,
-  "typescript.tsdk": "./common/node_modules/typescript/lib",
-  "tslint.autoFixOnSave": true
+  "typescript.tsdk": "./common/temp/node_modules/typescript/lib",
+  "tslint.autoFixOnSave": false
 }