The mail gem returns `UnstructuredField` when it fails to parse email addresses, but the `Receiver` always expects an `AddressList`.

.codeclimate.yml

@@ -0,0 +1,21 @@
+languages:
+   Ruby: true
+   JavaScript: true
+   Python: false
+   PHP: false
+
+exclude_paths:
+ - "app/assets/javascripts/defer/*"
+ - "app/assets/javascripts/ember-addons/*"
+ - "lib/autospec/*"
+ - "lib/es6_module_transpiler/*"
+ - "lib/highlight_js/*"
+ - "lib/import/*"
+ - "lib/javascripts/*"
+ - "lib/tasks/*"
+ - "lib/*.js"
+ - "public/*"
+ - "script/*"
+ - "spec/*"
+ - "test/*"
+ - "vendor/*"

.editorconfig

@@ -11,4 +11,4 @@ indent_style = space
 indent_size = 2
 
 [*.md]
-trim_trailing_whitespace = true
+trim_trailing_whitespace = false

.eslintignore

@@ -0,0 +1,17 @@
+app/assets/javascripts/env.js
+app/assets/javascripts/main_include_admin.js
+app/assets/javascripts/vendor.js
+app/assets/javascripts/locales/i18n.js
+app/assets/javascripts/ember-addons/
+app/assets/javascripts/discourse/lib/autosize.js.es6
+lib/javascripts/locale/
+lib/javascripts/messageformat.js
+lib/javascripts/moment.js
+lib/javascripts/moment_locale/
+lib/highlight_js/
+plugins/**/lib/javascripts/locale
+public/javascripts/
+vendor/
+test/javascripts/test_helper.js
+test/javascripts/fixtures
+test/javascripts/helpers/assertions.js

.eslintrc

@@ -0,0 +1,107 @@
+{
+  "env": {
+    "jasmine": true,
+    "node": true,
+    "mocha": true,
+    "browser": true,
+    "builtin": true
+  },
+  "parserOptions": {
+    "ecmaVersion": 7,
+    "sourceType": "module"
+  },
+  "globals":
+    {"Ember":true,
+    "jQuery":true,
+    "$":true,
+    "QUnit":true,
+    "RSVP":true,
+    "Discourse":true,
+    "Em":true,
+    "Handlebars":true,
+    "I18n":true,
+    "bootbox":true,
+    "moduleFor":true,
+    "moduleForComponent":true,
+    "Pretender":true,
+    "sandbox":true,
+    "controllerFor":true,
+    "test":true,
+    "visit":true,
+    "andThen":true,
+    "click":true,
+    "currentPath":true,
+    "currentRouteName":true,
+    "currentURL":true,
+    "fillIn":true,
+    "keyEvent":true,
+    "triggerEvent":true,
+    "count":true,
+    "exists":true,
+    "visible":true,
+    "invisible":true,
+    "asyncRender":true,
+    "selectKit":true,
+    "expandSelectKit":true,
+    "collapseSelectKit":true,
+    "selectKitSelectRowByValue":true,
+    "selectKitSelectRowByName":true,
+    "selectKitSelectRowByIndex":true,
+    "selectKitSelectNoneRow":true,
+    "selectKitFillInFilter":true,
+    "asyncTestDiscourse":true,
+    "fixture":true,
+    "find":true,
+    "sinon":true,
+    "moment":true,
+    "_":true,
+    "alert":true,
+    "define":true,
+    "require":true,
+    "requirejs":true,
+    "hasModule":true,
+    "Blob":true,
+    "File":true},
+  "rules": {
+    "block-scoped-var": 2,
+    "dot-notation": 0,
+    "eqeqeq": [
+      2,
+      "allow-null"
+    ],
+    "guard-for-in": 2,
+    "no-bitwise": 2,
+    "no-caller": 2,
+    "no-cond-assign": 0,
+    "no-debugger": 2,
+    "no-empty": 0,
+    "no-eval": 2,
+    "no-extend-native": 2,
+    "no-extra-parens": 0,
+    "no-inner-declarations": 2,
+    "no-irregular-whitespace": 2,
+    "no-iterator": 2,
+    "no-loop-func": 2,
+    "no-multi-str": 2,
+    "no-new": 2,
+    "no-plusplus": 0,
+    "no-proto": 2,
+    "no-script-url": 2,
+    "no-sequences": 2,
+    "no-shadow": 2,
+    "no-undef": 2,
+    "no-unused-vars": 2,
+    "no-with": 2,
+    "no-this-before-super": 2,
+    "semi": 2,
+    "strict": 0,
+    "valid-typeof": 2,
+    "wrap-iife": [
+      2,
+      "inside"
+    ],
+    "no-mixed-spaces-and-tabs": 2,
+    "no-trailing-spaces": 2
+  },
+  "parser": "babel-eslint"
+}

.gitattributes

@@ -1,11 +1,14 @@
 # Set default behaviour, in case users don't have core.autocrlf set.
 * text=auto
 
-# Explicitly declare text files we want to always be normalized and converted 
+# Treat email fixtures as binary files so CRLF are not converted to LF.
+*.eml binary
+
+# Explicitly declare text files we want to always be normalized and converted
 # to native line endings on checkout.
 *.yml text
 
-# Custom for Visual Studio, very unlikely, but lets keep it 
+# Custom for Visual Studio, very unlikely, but lets keep it
 *.cs     diff=csharp
 *.sln    merge=union
 *.csproj merge=union

.gitignore

@@ -4,22 +4,21 @@
 # or operating system, you probably want to add a global ignore instead:
 #   git config --global core.excludesfile ~/.gitignore_global
 
-tags
-
 .DS_Store
 ._.DS_Store
 dump.rdb
 
+bin/*
+
+data/
+
 .sass-cache/*
 public/csv/*
 public/plugins/*
 public/tombstone/*
 
 # Ignore bundler config
-/bin
 /.bundle
-/.vagrant
-/.vagrantfile
 /cache
 /coverage/*
 
@@ -31,22 +30,29 @@ config/discourse.pill
 config/discourse.conf
 
 # Ignore the default SQLite database and db dumps
+*.sql
+*.sql.gz
 /db/*.sqlite3
-/dbs/*.sql
-/dbs/*.sql.gz
 /db/structure.sql
+/db/schema.rb
 
 # Ignore all logfiles and tempfiles.
 /log/*.log
 /tmp
 /logfile
 log/
+bootsnap-load-path-cache
+bootsnap-compile-cache/
 
 # Ignore plugins except for the bundled ones.
 /plugins/*
 !/plugins/emoji/
 !/plugins/lazyYT/
 !/plugins/poll/
+!/plugins/discourse-details/
+!/plugins/discourse-nginx-performance-report
+!/plugins/discourse-narrative-bot
+!/plugins/discourse-presence
 /plugins/*/auto_generated/
 
 /spec/fixtures/plugins/my_plugin/auto_generated
@@ -57,6 +63,9 @@ log/
 # Ignore Eclipse .buildpath file
 /.buildpath
 
+# Ignore byebug history
+/.byebug_history
+
 # Ignore RubyMine settings
 /.idea
 
@@ -84,9 +93,6 @@ config/fog_credentials.yml
 script/download_db
 script/refresh_db
 
-# temp directory for chef (used to configure vagrant VM)
-chef/tmp/*
-
 # .procfile
 .procfile
 
@@ -104,3 +110,14 @@ config/version.rb
 bundler_stubs/*
 
 vendor/bundle/*
+*.db
+
+# ignore jetbrains ide file
+*.iml
+
+# vim swap
+*.swn
+
+# ignore nodejs files
+/node_modules
+/package-lock.json

.jshintignore

@@ -1,22 +0,0 @@
-app/assets/javascripts/env.js
-app/assets/javascripts/main_include.js
-app/assets/javascripts/main_include_admin.js
-app/assets/javascripts/preload_store.js
-app/assets/javascripts/pagedown_custom.js
-app/assets/javascripts/vendor.js
-app/assets/javascripts/locales/i18n.js
-app/assets/javascripts/defer/html-sanitizer-bundle.js
-app/assets/javascripts/discourse/lib/Markdown.Editor.js
-jsapp/lib/Markdown.Editor.js
-lib/javascripts/locale/
-lib/javascripts/messageformat.js
-lib/javascripts/moment.js
-lib/javascripts/moment_locale/
-lib/es6_module_transpiler/support/es6-module-transpiler.js
-public/javascripts/
-spec/phantom_js/smoke_test.js
-vendor/
-test/javascripts/helpers/
-test/javascripts/test_helper.js
-test/javascripts/test_helper.js
-

.jshintrc

@@ -1,83 +0,0 @@
-{
-  "predef":["Ember",
-    "jQuery",
-    "$",
-    "RSVP",
-    "Discourse",
-    "$LAB",
-    "Em",
-    "PreloadStore",
-    "Handlebars",
-    "I18n",
-    "bootbox",
-    "module",
-    "moduleFor",
-    "moduleForComponent",
-    "Pretender",
-    "sandbox",
-    "integration",
-    "controllerFor",
-    "test",
-    "ok",
-    "not",
-    "expect",
-    "equal",
-    "blank",
-    "present",
-    "visit",
-    "andThen",
-    "click",
-    "currentPath",
-    "currentRouteName",
-    "currentURL",
-    "fillIn",
-    "keyEvent",
-    "triggerEvent",
-    "count",
-    "exists",
-    "asyncTestDiscourse",
-    "fixture",
-    "find",
-    "sinon",
-    "moment",
-    "start",
-    "_",
-    "alert",
-    "containsInstance",
-    "parseHTML",
-    "deepEqual",
-    "notEqual",
-    "require",
-    "requirejs",
-    "hasModule",
-    "Blob",
-    "File"],
-  "node" : false,
-  "browser" : true,
-  "boss" : true,
-  "curly": false,
-  "debug": false,
-  "devel": false,
-  "eqeqeq": true,
-  "evil": true,
-  "forin": false,
-  "immed": false,
-  "laxbreak": false,
-  "newcap": true,
-  "noarg": true,
-  "noempty": false,
-  "nonew": false,
-  "nomen": false,
-  "onevar": false,
-  "plusplus": false,
-  "regexp": false,
-  "undef": true,
-  "unused": true,
-  "sub": true,
-  "strict": false,
-  "white": false,
-  "eqnull": true,
-  "quotmark": false,
-  "lastsemic": true,
-  "esnext": true
-}

.overcommit.yml

@@ -0,0 +1,23 @@
+# Use this file to configure the Overcommit hooks you wish to use. This will
+# extend the default configuration defined in:
+# https://github.com/brigade/overcommit/blob/master/config/default.yml
+#
+# At the topmost level of this YAML file is a key representing type of hook
+# being run (e.g. pre-commit, commit-msg, etc.). Within each type you can
+# customize each hook, such as whether to only run it on certain files (via
+# `include`), whether to only display output if it fails (via `quiet`), etc.
+#
+# For a complete list of hooks, see:
+# https://github.com/brigade/overcommit/tree/master/lib/overcommit/hook
+#
+# For a complete list of options that you can use to customize hooks, see:
+# https://github.com/brigade/overcommit#configuration
+
+PreCommit:
+  RuboCop:
+    enabled: true
+    command: ['bundle', 'exec', 'rubocop']
+  EsLint:
+    enabled: true
+    command: ['eslint', '--ext', '.es6', '-f', 'compact']
+    include: '**/*.es6'