From 3266eb31fade6cf5d7d71c5a1b6fb6bd2e05a84c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Dandelion=20Man=C3=A9?= <decentralion@dandelion.io>
Date: Fri, 27 Jul 2018 23:44:41 -0700
Subject: [PATCH] CLI takes repo strings as `owner/name` (#559)

Test plan:
```
$ yarn backend
$ node bin/sourcecred.js load sourcecred/sourcecred
```
---
 README.md                |  4 ++--
 scripts/deploy.sh        |  4 ++--
 src/cli/commands/load.js | 21 +++++++--------------
 3 files changed, 11 insertions(+), 18 deletions(-)

diff --git a/README.md b/README.md
index 85bb61f..42bc9d1 100644
--- a/README.md
+++ b/README.md
@@ -104,7 +104,7 @@ cd sourcecred
 yarn install
 yarn backend
 export SOURCECRED_GITHUB_TOKEN=YOUR_GITHUB_TOKEN
-node bin/sourcecred.js load REPO_OWNER REPO_NAME
+node bin/sourcecred.js load REPO_OWNER/REPO_NAME
 # this loads sourcecred data for a particular repository
 yarn start
 # then navigate to localhost:3000 in your browser
@@ -113,7 +113,7 @@ yarn start
 For example, if you wanted to look at cred for [ipfs/js-ipfs], you could run:
 ```
 $ export SOURCECRED_GITHUB_TOKEN=0000000000000000000000000000000000000000
-$ node bin/sourcecred.js load ipfs js-ipfs
+$ node bin/sourcecred.js load ipfs/js-ipfs
 ```
 
 replacing the big string of zeros with your actual token.
diff --git a/scripts/deploy.sh b/scripts/deploy.sh
index 1b5b9e9..69a34af 100755
--- a/scripts/deploy.sh
+++ b/scripts/deploy.sh
@@ -78,8 +78,8 @@ build_and_deploy() {
         yarn
         yarn backend
         yarn build
-        node ./bin/sourcecred.js load ipfs js-ipfs
-        node ./bin/sourcecred.js load sourcecred sourcecred
+        node ./bin/sourcecred.js load ipfs/js-ipfs
+        node ./bin/sourcecred.js load sourcecred/sourcecred
     )
 
     sourcecred_site="$(mktemp -d --suffix ".sourcecred-site")"
diff --git a/src/cli/commands/load.js b/src/cli/commands/load.js
index e6de73b..a5b2142 100644
--- a/src/cli/commands/load.js
+++ b/src/cli/commands/load.js
@@ -14,7 +14,7 @@ import {
   sourcecredDirectoryFlag,
 } from "../common";
 
-import {makeRepo} from "../../core/repo";
+import {repoToString, stringToRepo} from "../../core/repo";
 
 import {
   toJSON,
@@ -31,14 +31,9 @@ export default class PluginGraphCommand extends Command {
 
   static args = [
     {
-      name: "repo_owner",
+      name: "repo",
       required: true,
-      description: "owner of the GitHub repository for which to fetch data",
-    },
-    {
-      name: "repo_name",
-      required: true,
-      description: "name of the GitHub repository for which to fetch data",
+      description: "the GitHub repo to load, represented as OWNER/NAME",
     },
   ];
 
@@ -61,7 +56,7 @@ export default class PluginGraphCommand extends Command {
 
   async run() {
     const {
-      args: {repo_owner: owner, repo_name: name},
+      args,
       flags: {
         "github-token": githubToken,
         "sourcecred-directory": basedir,
@@ -69,7 +64,7 @@ export default class PluginGraphCommand extends Command {
         plugin,
       },
     } = this.parse(PluginGraphCommand);
-    const repo = makeRepo(owner, name);
+    const repo = stringToRepo(args.repo);
     if (!plugin) {
       loadAllPlugins({
         basedir,
@@ -101,8 +96,7 @@ function loadAllPlugins({basedir, repo, githubToken, maxOldSpaceSize}) {
         `--max_old_space_size=${maxOldSpaceSize}`,
         "./bin/sourcecred.js",
         "load",
-        repo.owner,
-        repo.name,
+        repoToString(repo),
         "--plugin",
         pluginName,
         "--github-token",
@@ -123,8 +117,7 @@ function loadPlugin({basedir, plugin, repo, githubToken}) {
   const outputDirectory = path.join(
     basedir,
     "data",
-    repo.owner,
-    repo.name,
+    repoToString(repo),
     plugin
   );
   mkdirp.sync(outputDirectory);