Merge branch 'master' into sg-clean-up-dependency-insert

Author: jtgeibel

.rustfmt.toml

@@ -1,11 +1,4 @@
-fn_args_layout = "Block"
-array_layout = "Block"
-control_style = "Rfc"
-where_style = "Rfc"
-generics_indent = "Block"
-fn_call_style = "Block"
 combine_control_expr = true
-fn_args_paren_newline = false
 max_width = 100
 error_on_line_overflow = false
 write_mode = "Overwrite"

.travis.yml

@@ -21,9 +21,7 @@ before_install:
   - nvm install 8
 
 install:
-  - pip install 'travis-cargo<0.2' --user && export PATH=$HOME/.local/bin:$PATH
-  - npm install
-  - cargo install --force diesel_cli --vers 0.16.0 --debug --no-default-features --features postgres && export PATH=$HOME/.cargo/bin:$PATH
+  - cargo install --force diesel_cli --vers 1.0.0 --no-default-features --features postgres && export PATH=$HOME/.cargo/bin:$PATH
 
 before_script:
   - diesel database setup
@@ -44,16 +42,17 @@ matrix:
   allow_failures:
     - rust: nightly
   include:
-  - rust: nightly-2017-11-07
+  - rust: nightly-2017-12-28
     script:
-    - cargo install --force rustfmt-nightly --vers 0.2.15
-    - cargo install --force clippy --vers 0.0.169
+    - cargo install --force rustfmt-nightly --vers 0.3.4
     - cargo fmt -- --write-mode=diff
+    - cargo install --force clippy --vers 0.0.177
     - cargo clippy
   - rust: stable
     script:
     - cargo build
     - cargo test
+    - npm install
     - npm test
   - rust: beta
     script:
@@ -68,6 +67,7 @@ env:
   global:
     - DATABASE_URL=postgres://postgres:@localhost/cargo_registry_test
     - TEST_DATABASE_URL=postgres://postgres:@localhost/cargo_registry_test
+    - CARGO_TARGET_DIR=target
 
 notifications:
   email:

Cargo.lock

@@ -84,3 +84,7 @@ tokio-service = "0.1"
 dotenv = "0.10"
 diesel = { version = "1.0.0-beta1", features = ["postgres"] }
 diesel_migrations = { version = "1.0.0-beta1", features = ["postgres"] }
+
+# Remove once cookie depends on ring >= 0.13.0
+[patch.crates-io]
+ring = { git = "https://github.com/SergioBenitez/ring", branch = "v0.11" }

Cargo.toml

@@ -21,11 +21,7 @@ export default Controller.extend(EKMixin, {
     }),
 
     actions: {
-        search(q) {
-            if (q !== undefined) {
-                this.set('searchQuery', q);
-            }
-
+        search() {
             this.transitionToRoute('search', {
                 queryParams: {
                     q: this.get('searchQuery'),

app/controllers/application.js

@@ -3,12 +3,10 @@ import { computed } from '@ember/object';
 import { alias, bool, readOnly } from '@ember/object/computed';
 import { inject as service } from '@ember/service';
 
-import { task, timeout } from 'ember-concurrency';
+import { task } from 'ember-concurrency';
 
 import PaginationMixin from '../mixins/pagination';
 
-const DEBOUNCE_MS = 250;
-
 export default Controller.extend(PaginationMixin, {
     search: service(),
     queryParams: ['q', 'page', 'per_page', 'sort'],
@@ -42,13 +40,10 @@ export default Controller.extend(PaginationMixin, {
     hasItems: bool('totalItems'),
 
     dataTask: task(function* (params) {
-        // debounce the search query
-        yield timeout(DEBOUNCE_MS);
-
         if (params.q !== null) {
             params.q = params.q.trim();
         }
 
         return yield this.store.query('crate', params);
-    }).restartable(),
+    }).drop(),
 });

app/controllers/search.js

@@ -21,7 +21,7 @@
         <input type="text" class="search" name="q" id="cargo-desktop-search"
                 placeholder="Click or press 'S' to search..."
                 value={{searchQuery}}
-                oninput={{action "search" value="target.value"}}
+                oninput={{action (mut searchQuery) value="target.value"}}
                 autofocus="autofocus"
                 tabindex="1"
                 required
@@ -106,7 +106,7 @@
     <input type="text" class="search" name="q"
             placeholder="Search"
             value={{searchQuery}}
-            oninput={{action "search" value="target.value"}}
+            oninput={{action (mut searchQuery) value="target.value"}}
             autocorrect="off"
             tabindex="1"
             required>

app/templates/application.hbs

@@ -11,7 +11,7 @@
                 </h2>
             </div>
             {{#user-link user=model.team data-test-github-link=true}}
-                <img alt="GitHub profile" title="GitHub profile" src="/assets/GitHub-Mark-32px.png"/>
+                <img alt="GitHub profile" title="GitHub profile" src="/assets/GitHub-Mark.svg"/>
             {{/user-link}}
         </div>
     </div>

app/templates/team.hbs

@@ -4,7 +4,7 @@
         {{ model.user.login }}
     </h1>
     {{#user-link user=model.user data-test-user-link=true}}
-      <img alt="GitHub profile" title="GitHub profile" src="/assets/GitHub-Mark-32px.png"/>
+      <img alt="GitHub profile" title="GitHub profile" src="/assets/GitHub-Mark.svg"/>
     {{/user-link}}
 </div>
 

app/templates/user.hbs

@@ -29,8 +29,12 @@ pub enum Badge {
         repository: String,
         branch: Option<String>,
     },
-    IsItMaintainedIssueResolution { repository: String },
-    IsItMaintainedOpenIssues { repository: String },
+    IsItMaintainedIssueResolution {
+        repository: String,
+    },
+    IsItMaintainedOpenIssues {
+        repository: String,
+    },
     Codecov {
         repository: String,
         branch: Option<String>,
@@ -41,7 +45,9 @@ pub enum Badge {
         branch: Option<String>,
         service: Option<String>,
     },
-    Maintenance { status: MaintenanceStatus },
+    Maintenance {
+        status: MaintenanceStatus,
+    },
 }
 
 #[derive(Debug, PartialEq, Clone, Copy, Deserialize, Serialize)]

public/assets/GitHub-Mark-32px.png

@@ -39,8 +39,7 @@ fn delete(conn: &PgConnection) {
     let krate = Crate::by_name(&name).first::<Crate>(conn).unwrap();
     print!(
         "Are you sure you want to delete {} ({}) [y/N]: ",
-        name,
-        krate.id
+        name, krate.id
     );
     io::stdout().flush().unwrap();
     let mut line = String::new();

public/assets/GitHub-Mark.svg

@@ -50,9 +50,7 @@ fn delete(conn: &PgConnection) {
         .unwrap();
     print!(
         "Are you sure you want to delete {}#{} ({}) [y/N]: ",
-        name,
-        version,
-        v.id
+        name, version, v.id
     );
     io::stdout().flush().unwrap();
     let mut line = String::new();

src/badge.rs

@@ -132,8 +132,7 @@ fn main() {
             let config = config.clone();
             version.record_readme_rendering(&conn).expect(&format!(
                 "[{}-{}] Couldn't record rendering time",
-                krate_name,
-                version.num
+                krate_name, version.num
             ));
             let handle = thread::spawn(move || {
                 println!("[{}-{}] Rendering README...", krate_name, version.num);
@@ -155,8 +154,7 @@ fn main() {
                     )
                     .expect(&format!(
                         "[{}-{}] Couldn't upload file to S3",
-                        krate_name,
-                        version.num
+                        krate_name, version.num
                     ));
             });
             tasks.push(handle);
@@ -182,8 +180,7 @@ fn get_readme(config: &Config, version: &Version, krate_name: &str) -> Option<St
     let date = Utc::now().to_rfc2822();
     let url = Url::parse(&location).expect(&format!(
         "[{}-{}] Couldn't parse crate URL",
-        krate_name,
-        version.num
+        krate_name, version.num
     ));
 
     let mut headers = List::new();
@@ -206,9 +203,7 @@ fn get_readme(config: &Config, version: &Version, krate_name: &str) -> Option<St
         if let Err(err) = req.perform() {
             println!(
                 "[{}-{}] Unable to fetch crate: {}",
-                krate_name,
-                version.num,
-                err
+                krate_name, version.num, err
             );
             return None;
         }
@@ -217,31 +212,26 @@ fn get_readme(config: &Config, version: &Version, krate_name: &str) -> Option<St
         let response = String::from_utf8_lossy(&response);
         println!(
             "[{}-{}] Failed to get a 200 response: {}",
-            krate_name,
-            version.num,
-            response
+            krate_name, version.num, response
         );
         return None;
     }
     let reader = Cursor::new(response);
     let reader = GzDecoder::new(reader).expect(&format!(
         "[{}-{}] Invalid gzip header",
-        krate_name,
-        version.num
+        krate_name, version.num
     ));
     let mut archive = Archive::new(reader);
     let mut entries = archive.entries().expect(&format!(
         "[{}-{}] Invalid tar archive entries",
-        krate_name,
-        version.num
+        krate_name, version.num
     ));
     let manifest: Manifest = {
         let path = format!("{}-{}/Cargo.toml", krate_name, version.num);
         let contents = find_file_by_path(&mut entries, Path::new(&path), version, krate_name);
         toml::from_str(&contents).expect(&format!(
             "[{}-{}] Syntax error in manifest file",
-            krate_name,
-            version.num
+            krate_name, version.num
         ))
     };
     if manifest.package.readme.is_none() {
@@ -265,8 +255,7 @@ fn get_readme(config: &Config, version: &Version, krate_name: &str) -> Option<St
             manifest.package.repository.as_ref().map(|e| &**e),
         ).expect(&format!(
             "[{}-{}] Couldn't render README",
-            krate_name,
-            version.num
+            krate_name, version.num
         ))
     };
     return Some(rendered);
@@ -315,8 +304,7 @@ fn find_file_by_path<R: Read>(
     let mut contents = String::new();
     file.read_to_string(&mut contents).expect(&format!(
         "[{}-{}] Couldn't read file contents",
-        krate_name,
-        version.num
+        krate_name, version.num
     ));
     contents
 }

src/bin/delete-crate.rs

@@ -64,8 +64,7 @@ fn transfer(conn: &PgConnection) {
 
     println!(
         "Are you sure you want to transfer crates from {} to {}",
-        from.gh_login,
-        to.gh_login
+        from.gh_login, to.gh_login
     );
     get_confirm("continue");
 

src/bin/delete-version.rs

@@ -57,9 +57,9 @@ fn categories_from_toml(
     let mut result = vec![];
 
     for (slug, details) in categories {
-        let details = details.as_table().chain_error(|| {
-            internal(&format_args!("category {} was not a TOML table", slug))
-        })?;
+        let details = details
+            .as_table()
+            .chain_error(|| internal(&format_args!("category {} was not a TOML table", slug)))?;
 
         let category = Category::from_parent(
             slug,

src/bin/render-readmes.rs

@@ -88,11 +88,9 @@ impl Category {
                 .collect();
             let crate_categories = categories
                 .iter()
-                .map(|c| {
-                    CrateCategory {
-                        category_id: c.id,
-                        crate_id: krate.id,
-                    }
+                .map(|c| CrateCategory {
+                    category_id: c.id,
+                    crate_id: krate.id,
                 })
                 .collect::<Vec<_>>();
 
@@ -137,9 +135,7 @@ impl Category {
              WHERE split_part(c.slug, '::', 1) = c.slug
              GROUP BY c.id
              {} LIMIT {} OFFSET {}",
-            sort_sql,
-            limit,
-            offset
+            sort_sql, limit, offset
         ))).load(conn)
     }
 

src/bin/transfer-crates.rs

@@ -97,9 +97,7 @@ pub fn add_dependencies(
         .map(|dep| {
             let krate = Crate::by_name(&dep.name)
                 .first::<Crate>(&*conn)
-                .map_err(|_| {
-                    human(&format_args!("no known crate named `{}`", &*dep.name))
-                })?;
+                .map_err(|_| human(&format_args!("no known crate named `{}`", &*dep.name)))?;
             if dep.version_req == semver::VersionReq::parse("*").unwrap() {
                 return Err(human(
                     "wildcard (`*`) dependency constraints are not allowed \

src/boot/categories.rs

@@ -72,8 +72,7 @@ pub fn parse_github_response<'de, 'a: 'de, T: Deserialize<'de>>(
             return Err(internal(&format_args!(
                 "didn't get a 200 result from \
                  github, got {} with: {}",
-                n,
-                resp
+                n, resp
             )));
         }
     }