Auto merge of #3132 - Turbo87:author, r=hi-rustin

Remove "Authors" section from crate details page

see rust-lang/rfcs#3052 😉

r? `@pietroalbini`

app/components/crate-sidebar.hbs

@@ -103,15 +103,6 @@
         {{/each}}
       </ul>
     </div>
-
-    <div>
-      <h3>Authors</h3>
-      <ul>
-        {{#each @version.authorNames as |author|}}
-          <li>{{ format-email author }}</li>
-        {{/each}}
-      </ul>
-    </div>
   </div>
 
   <div local-class='bottom'>

app/models/version.js

@@ -25,7 +25,6 @@ export default class Version extends Model {
   @belongsTo('crate', { async: false }) crate;
 
   @belongsTo('user', { async: false }) published_by;
-  @hasMany('users', { async: true }) authors;
   @hasMany('dependency', { async: true }) dependencies;
   @hasMany('version-download', { async: true }) version_downloads;
 
@@ -97,15 +96,6 @@ export default class Version extends Model {
       .map(name => ({ name, isDefault: defaultFeatures.includes(name), dependencies: features[name] }));
   }
 
-  @alias('loadAuthorsTask.last.value') authorNames;
-
-  @(task(function* () {
-    // trigger the async relationship to load the content
-    let authors = yield this.authors;
-    return authors.meta.names;
-  }).keepLatest())
-  loadAuthorsTask;
-
   @alias('loadDepsTask.last.value.normal') normalDependencies;
   @alias('loadDepsTask.last.value.build') buildDependencies;
   @alias('loadDepsTask.last.value.dev') devDependencies;

app/routes/crate/version.js

@@ -32,10 +32,6 @@ export default class VersionRoute extends Route {
   setupController(controller, model) {
     super.setupController(...arguments);
 
-    if (!model.version.authorNames) {
-      model.version.loadAuthorsTask.perform();
-    }
-
     controller.loadReadmeTask.perform().catch(() => {
       // ignored
     });