teachers_pet 0.3.2 → 0.3.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 813064fc57381e80d03d7c3743cf2b77a295e50c
-  data.tar.gz: 8209de5e067d2ae7c97259a9d991dfd49eba4058
+  metadata.gz: 237378cc3ef99f43b205dd00666e73b31d0fd08c
+  data.tar.gz: 6ae28a5c307127776aef86f6f40570fa38b6603f
 SHA512:
-  metadata.gz: 41e6db46507899a90c0aff59d168b2c4f8091c43fd76fdd5580bddbea563a56b18f353013ab46310ea94fdbc8e50dcfe41210def8942a9eba9beaae21f0b2579
-  data.tar.gz: 96e44bd2c0c615688c21803076cc4640dfc8de7d830a75d1c2d63ac025c8bc90e0c42621d73b509187109aac8ace30405ac134b42934a636ac531e280d4c85be
+  metadata.gz: 6045ed09c10c41e2167b5f87223d2a46b91e99d2473f73d1b0bff31e17683a830328e7833596d6d2aace1c4e859049b5d66e42a6d961a19bbf8260e5a02ad3ef
+  data.tar.gz: b23a23912a4c70d10a47d2092693d19272173ea1a3cc65fd7f352d046ec5b119fcc4fe49976550cd2d5fc4e6e95c19c426d4e1fefa4299d11628649b5501cd3b

data/README.md

@@ -92,9 +92,13 @@ You may need to give other people access to various repositories using teams –
 
 When using the [sandboxing](https://education.github.com/guide/sandboxing) setup, you will need to create the repositories for the students.  For each assignment, use the `create_repos` action to create a repository for each student.  The repositories are technically created per team, but if you use `create_student_teams` first, then there will be one team per student.
 
+### Forks
+
+If you need to grab the list of users who have forked a particular repository – e.g. to use with another command – you can run the `forks` command, and the results will be written to a file.
+
 ### Collaborator access
 
-Give [collaborator access](https://help.github.com/articles/what-are-the-different-access-permissions#collaborator) to everyone who has forked your repository using `fork_collab`.  Mostly useful for GitHub demonstrations, where the students can quickly be added to a repository without having to worry about collecting usernames.
+Give [collaborator access](https://help.github.com/articles/what-are-the-different-access-permissions#collaborator) to everyone who has forked your repository using `add_collaborators`.  Mostly useful for GitHub demonstrations, where usernames can quickly be collected via `forks`, and then the students can be quickly given access to a repository.
 
 ### Pushing starter files
 
@@ -104,10 +108,14 @@ When creating repositories for students, you will often want to include boilerpl
 
 After running `create_repos`, instructors can open issues in student repos as a way to list requirements of the assignment, goals, or instructions for patching, using the `open_issue` command.
 
-### Pulling repositories for grading
+### Clone repositories for grading
 
 When grading, use the `clone_repos` command to clone all the repositories in the organization that match the username-repository naming scheme that is generated when `create_repos` is run.
 
+### Merge all open pull requests
+
+When running a GitHub workshop, it's nice to be able to merge a bunch of pull requests on a particular repository all at once. `merge_pull_requests` will handle this for you.
+
 ## Related projects
 
 * https://education.github.com/guide

data/lib/teachers_pet/actions/add_collaborators.rb

@@ -0,0 +1,21 @@
+module TeachersPet
+  module Actions
+    class AddCollaborators < Base
+      def repository
+        self.options[:repository]
+      end
+
+      def run
+        self.init_client
+
+        members = self.read_members_file
+        members.each do |login|
+          unless self.options[:dry_run]
+            result = self.client.add_collab(self.repository, login)
+          end
+          puts "#{login} - #{result}"
+        end
+      end
+    end
+  end
+end

data/lib/teachers_pet/actions/add_to_team.rb

@@ -1,12 +1,6 @@
 module TeachersPet
   module Actions
     class AddToTeam < Base
-      def read_members_file
-        file = self.options[:members]
-        puts "Loading members to add:"
-        read_file(file).keys
-      end
-
       def team_name
         file = self.options[:members]
         File.basename(file, File.extname(file))

data/lib/teachers_pet/actions/base.rb

@@ -73,6 +73,12 @@ module TeachersPet
         puts "Loading students:"
         read_file(student_file)
       end
+
+      def read_members_file
+        file = self.options[:members]
+        puts "Loading members to add:"
+        read_file(file).keys
+      end
     end
   end
 end

data/lib/teachers_pet/actions/forks.rb

@@ -0,0 +1,31 @@
+require 'csv'
+
+module TeachersPet
+  module Actions
+    class Forks < Base
+      def repository
+        self.options[:repository]
+      end
+
+      def get_forks
+        self.client.forks(self.repository)
+      end
+
+      def run
+        self.init_client
+        forks = self.get_forks
+
+        CSV.open(self.options[:output], 'wb') do |csv|
+          forks.each do |fork|
+            login = fork.owner.login
+            if fork.owner.type == "User"
+              csv << [login]
+            else
+              puts "Ignoring organization: @#{login}"
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/teachers_pet/actions/merge_pull_requests.rb

@@ -0,0 +1,18 @@
+module TeachersPet
+  module Actions
+    class MergePullRequests < Base
+      def run
+        repository = self.options[:repository]
+
+        self.init_client
+
+        open_pull_requests = self.client.pull_requests(repository, state: 'open')
+        open_pull_requests.each do |pr|
+          print "Merging #{pr.html_url}..."
+          client.merge_pull_request(repository, pr.number)
+          puts "done"
+        end
+      end
+    end
+  end
+end

data/lib/teachers_pet/commands/add_collaborators.rb

@@ -0,0 +1,13 @@
+module TeachersPet
+  class Cli
+    option :repository, required: true, banner: 'OWNER/REPO'
+    option :members, required: true, banner: 'PATH', desc: "The path to the file containing the list of usernames to add."
+    option :dry_run, type: :boolean, default: false
+    common_options
+
+    desc "add_collaborators", "Give collaborator access to each provided user."
+    def add_collaborators
+      TeachersPet::Actions::AddCollaborators.new(options).run
+    end
+  end
+end

data/lib/teachers_pet/commands/add_to_team.rb

@@ -1,6 +1,7 @@
 module TeachersPet
   class Cli
     option :organization, required: true
+    # TODO make team name configurable
     option :members, required: true, banner: 'PATH', desc: "The path to the file containing the list of members to add. The filename will be used as the name of the team, e.g. `path/to/instructors.csv` will use the 'instructors' team."
     common_options
 

data/lib/teachers_pet/commands/forks.rb

@@ -0,0 +1,12 @@
+module TeachersPet
+  class Cli
+    option :repository, required: true, banner: 'OWNER/REPO'
+    option :output, banner: 'PATH', default: 'students.csv'
+    common_options
+
+    desc "forks", "Pull the list of users who have forked a particular repository."
+    def forks
+      TeachersPet::Actions::Forks.new(options).run
+    end
+  end
+end

data/lib/teachers_pet/commands/merge_pull_requests.rb

@@ -0,0 +1,11 @@
+module TeachersPet
+  class Cli
+    option :repository, required: true, banner: 'OWNER/REPO'
+    common_options
+
+    desc 'merge_pull_requests', "Merges all open pull requests on a particular repository"
+    def merge_pull_requests
+      TeachersPet::Actions::MergePullRequests.new(options).run
+    end
+  end
+end

data/lib/teachers_pet/version.rb

@@ -1,3 +1,3 @@
 module TeachersPet
-  VERSION = '0.3.2'
+  VERSION = '0.3.3'
 end

data/spec/commands/add_collaborators_spec.rb

@@ -0,0 +1,84 @@
+require 'spec_helper'
+
+describe 'add_collaborators' do
+  include CommandHelpers
+
+  def stub_add_students
+    student_usernames.map do |username|
+      stub_request(:put, "https://testteacher:abc123@api.github.com/repos/testorg/testrepo/collaborators/#{username}")
+    end
+  end
+
+  context 'through CLI' do
+    it "requires the repository be specified" do
+      expect {
+        teachers_pet(:add_collaborators)
+      }.to raise_error(Thor::RequiredArgumentMissingError, /--repository/)
+    end
+
+    it "passes the options to the action" do
+      expect_to_be_run_with(TeachersPet::Actions::AddCollaborators,
+        'api' => 'https://api.github.com/',
+        'dry_run' => false,
+        'members' => students_list_fixture_path,
+        'password' => 'abc123',
+        'repository' => 'testorg/testrepo',
+        'username' => ENV['USER'],
+        'web' => 'https://www.github.com/'
+      )
+
+      teachers_pet(:add_collaborators,
+        repository: 'testorg/testrepo',
+        members: students_list_fixture_path,
+        password: 'abc123'
+      )
+    end
+
+    it "succeeds for basic auth" do
+      request_stubs = student_usernames.map do |username|
+        stub_request(:put, "https://testteacher:abc123@api.github.com/repos/testorg/testrepo/collaborators/#{username}")
+      end
+
+      teachers_pet(:add_collaborators,
+        repository: 'testorg/testrepo',
+        members: students_list_fixture_path,
+        username: 'testteacher',
+        password: 'abc123'
+      )
+
+      request_stubs.each do |request_stub|
+        expect(request_stub).to have_been_requested.once
+      end
+    end
+
+    it "succeeds for OAuth" do
+      request_stubs = student_usernames.map do |username|
+        stub_request(:put, "https://api.github.com/repos/testorg/testrepo/collaborators/#{username}").with(headers: {'Authorization'=>'token tokentokentoken'})
+      end
+
+      teachers_pet(:add_collaborators,
+        repository: 'testorg/testrepo',
+        members: students_list_fixture_path,
+        token: 'tokentokentoken'
+      )
+
+      request_stubs.each do |request_stub|
+        expect(request_stub).to have_been_requested.once
+      end
+    end
+
+    it "prints the users on a dry run" do
+      output = capture(:stdout) do
+        teachers_pet(:add_collaborators,
+          repository: 'testorg/testrepo',
+          members: students_list_fixture_path,
+          username: 'testteacher',
+          password: 'abc123',
+          dry_run: true
+        )
+      end
+
+      expect(output).to include('teststudent1')
+    end
+  end
+end

data/spec/commands/{fork_collab_spec.rb → forks_spec.rb}

@@ -1,30 +1,60 @@
 require 'spec_helper'
 
-describe 'fork_collab' do
+describe 'forks' do
   include CommandHelpers
 
+  let(:members_file){ 'students.csv' }
+
+  after do
+    FileUtils.rm_f(members_file)
+  end
+
   context 'through CLI' do
     it "requires the repository be specified" do
       expect {
-        teachers_pet(:fork_collab)
+        teachers_pet(:forks)
       }.to raise_error(Thor::RequiredArgumentMissingError, /--repository/)
     end
 
     it "passes the options to the action" do
-      expect_to_be_run_with(TeachersPet::Actions::ForkCollab,
+      expect_to_be_run_with(TeachersPet::Actions::Forks,
         'api' => 'https://api.github.com/',
-        'dry_run' => false,
+        'output' => members_file,
         'password' => 'abc123',
         'repository' => 'testorg/testrepo',
         'username' => ENV['USER'],
         'web' => 'https://www.github.com/'
       )
-      teachers_pet(:fork_collab, repository: 'testorg/testrepo', password: 'abc123')
+      teachers_pet(:forks, repository: 'testorg/testrepo', password: 'abc123')
+    end
+
+    context "with a different filename" do
+      let(:members_file){ './users.txt' }
+
+      it "writes to that file" do
+        request_stub = stub_get_json('https://testteacher:abc123@api.github.com/repos/testorg/testrepo/forks?per_page=100', [
+          {
+            owner: {
+              login: 'teststudent',
+              type: 'User'
+            }
+          }
+        ])
+
+        teachers_pet(:forks,
+          repository: 'testorg/testrepo',
+          output: members_file,
+          username: 'testteacher',
+          password: 'abc123'
+        )
+
+        expect(File.read(members_file)).to eq("teststudent\n")
+        expect(request_stub).to have_been_requested.once
+      end
     end
 
     it "succeeds for basic auth" do
-      request_stubs = []
-      request_stubs << stub_get_json('https://testteacher:abc123@api.github.com/repos/testorg/testrepo/forks?per_page=100', [
+      request_stub = stub_get_json('https://testteacher:abc123@api.github.com/repos/testorg/testrepo/forks?per_page=100', [
         {
           owner: {
             login: 'teststudent',
@@ -32,22 +62,19 @@ describe 'fork_collab' do
           }
         }
       ])
-      request_stubs << stub_request(:put, 'https://testteacher:abc123@api.github.com/repos/testorg/testrepo/collaborators/teststudent')
 
-      teachers_pet(:fork_collab,
+      teachers_pet(:forks,
         repository: 'testorg/testrepo',
         username: 'testteacher',
         password: 'abc123'
       )
 
-      request_stubs.each do |request_stub|
-        expect(request_stub).to have_been_requested.once
-      end
+      expect(File.read(members_file)).to eq("teststudent\n")
+      expect(request_stub).to have_been_requested.once
     end
 
     it "succeeds for OAuth" do
-      request_stubs = []
-      request_stubs << stub_get_json('https://api.github.com/repos/testorg/testrepo/forks?per_page=100', [
+      request_stub = stub_get_json('https://api.github.com/repos/testorg/testrepo/forks?per_page=100', [
         {
           owner: {
             login: 'teststudent',
@@ -56,39 +83,12 @@ describe 'fork_collab' do
         }
       ]).with(headers: {'Authorization' => 'token tokentokentoken'})
 
-      request_stubs << stub_request(:put, 'https://api.github.com/repos/testorg/testrepo/collaborators/teststudent').
-        with(headers: {'Authorization' => 'token tokentokentoken'})
-
-      teachers_pet(:fork_collab,
+      teachers_pet(:forks,
         repository: 'testorg/testrepo',
         token: 'tokentokentoken'
       )
 
-      request_stubs.each do |request_stub|
-        expect(request_stub).to have_been_requested.once
-      end
-    end
-
-    it "prints the users on a dry run" do
-      request_stub = stub_get_json('https://testteacher:abc123@api.github.com/repos/testorg/testrepo/forks?per_page=100', [
-        {
-          owner: {
-            login: 'teststudent',
-            type: 'User'
-          }
-        }
-      ])
-
-      output = capture(:stdout) do
-        teachers_pet(:fork_collab,
-          repository: 'testorg/testrepo',
-          username: 'testteacher',
-          password: 'abc123',
-          dry_run: true
-        )
-      end
-
-      expect(output).to include('teststudent')
+      expect(File.read(members_file)).to eq("teststudent\n")
       expect(request_stub).to have_been_requested.once
     end
   end

data/spec/commands/merge_pull_requests_spec.rb

@@ -0,0 +1,29 @@
+require 'spec_helper'
+
+describe 'merge_pull_requests' do
+  include CommandHelpers
+
+  it "merges all open pull requests in a particular repository" do
+    request_stubs = []
+    request_stubs << stub_get_json("https://testteacher:abc123@api.github.com/repos/testorg/testrepo/pulls?per_page=100&state=open", [
+      { id: 1, html_url: "https://github.com/testorg/testrepo/pull/1", number: 1 },
+      { id: 2, html_url: "https://github.com/testorg/testrepo/pull/2", number: 2 },
+      { id: 3, html_url: "https://github.com/testorg/testrepo/pull/3", number: 3 }
+    ])
+
+    (1..3).to_a.each do |i|
+      request_stubs << stub_request(:put, "https://testteacher:abc123@api.github.com/repos/testorg/testrepo/pulls/#{i}/merge")
+    end
+
+    teachers_pet(:merge_pull_requests,
+      repository: 'testorg/testrepo',
+
+      username: 'testteacher',
+      password: 'abc123'
+    )
+
+    request_stubs.each do |request_stub|
+      expect(request_stub).to have_been_requested.once
+    end
+  end
+end

metadata

@@ -1,153 +1,153 @@
 --- !ruby/object:Gem::Specification
 name: teachers_pet
 version: !ruby/object:Gem::Version
-  version: 0.3.2
+  version: 0.3.3
 platform: ruby
 authors:
 - John Britton
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-06-18 00:00:00.000000000 Z
+date: 2014-06-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '4.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '4.0'
 - !ruby/object:Gem::Dependency
   name: octokit
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 3.1.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 3.1.0
 - !ruby/object:Gem::Dependency
   name: require_all
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 1.3.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 1.3.2
 - !ruby/object:Gem::Dependency
   name: thor
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 0.19.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 0.19.1
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.5'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.5'
 - !ruby/object:Gem::Dependency
   name: guard-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '4.2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '4.2'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '3.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '3.0'
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 0.7.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 0.7.1
 - !ruby/object:Gem::Dependency
   name: webmock
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.17'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.17'
 description: 
@@ -157,9 +157,9 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
-- .rspec
-- .travis.yml
+- ".gitignore"
+- ".rspec"
+- ".travis.yml"
 - CONTRIBUTING.md
 - Gemfile
 - Guardfile
@@ -168,32 +168,38 @@ files:
 - Rakefile
 - bin/teachers_pet
 - lib/teachers_pet.rb
+- lib/teachers_pet/actions/add_collaborators.rb
 - lib/teachers_pet/actions/add_to_team.rb
 - lib/teachers_pet/actions/base.rb
 - lib/teachers_pet/actions/clone_repos.rb
 - lib/teachers_pet/actions/create_repos.rb
 - lib/teachers_pet/actions/create_student_teams.rb
-- lib/teachers_pet/actions/fork_collab.rb
+- lib/teachers_pet/actions/forks.rb
+- lib/teachers_pet/actions/merge_pull_requests.rb
 - lib/teachers_pet/actions/open_issue.rb
 - lib/teachers_pet/actions/push_files.rb
 - lib/teachers_pet/cli.rb
 - lib/teachers_pet/client_decorator.rb
+- lib/teachers_pet/commands/add_collaborators.rb
 - lib/teachers_pet/commands/add_to_team.rb
 - lib/teachers_pet/commands/clone_repos.rb
 - lib/teachers_pet/commands/create_repos.rb
 - lib/teachers_pet/commands/create_student_teams.rb
-- lib/teachers_pet/commands/fork_collab.rb
+- lib/teachers_pet/commands/forks.rb
+- lib/teachers_pet/commands/merge_pull_requests.rb
 - lib/teachers_pet/commands/open_issue.rb
 - lib/teachers_pet/commands/push_files.rb
 - lib/teachers_pet/configuration.rb
 - lib/teachers_pet/version.rb
 - spec/actions/base_spec.rb
 - spec/cli_spec.rb
+- spec/commands/add_collaborators_spec.rb
 - spec/commands/add_to_team_spec.rb
 - spec/commands/clone_repos_spec.rb
 - spec/commands/create_repos_spec.rb
 - spec/commands/create_student_teams_spec.rb
-- spec/commands/fork_collab_spec.rb
+- spec/commands/forks_spec.rb
+- spec/commands/merge_pull_requests_spec.rb
 - spec/commands/open_issue_spec.rb
 - spec/commands/push_files_spec.rb
 - spec/fixtures/empty
@@ -216,28 +222,30 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: 1.9.3
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.3.0
+rubygems_version: 2.2.2
 signing_key: 
 specification_version: 4
 summary: Command line tools to help teachers use GitHub in their classrooms
 test_files:
 - spec/actions/base_spec.rb
 - spec/cli_spec.rb
+- spec/commands/add_collaborators_spec.rb
 - spec/commands/add_to_team_spec.rb
 - spec/commands/clone_repos_spec.rb
 - spec/commands/create_repos_spec.rb
 - spec/commands/create_student_teams_spec.rb
-- spec/commands/fork_collab_spec.rb
+- spec/commands/forks_spec.rb
+- spec/commands/merge_pull_requests_spec.rb
 - spec/commands/open_issue_spec.rb
 - spec/commands/push_files_spec.rb
 - spec/fixtures/empty

data/lib/teachers_pet/actions/fork_collab.rb

@@ -1,33 +0,0 @@
-module TeachersPet
-  module Actions
-    class ForkCollab < Base
-      def repository
-        self.options[:repository]
-      end
-
-      def get_forks
-        self.client.forks(self.repository)
-      end
-
-      def promote
-        self.init_client
-        forks = self.get_forks
-        forks.each do |fork|
-          login = fork.owner.login
-          if fork.owner.type == "User"
-            unless self.options[:dry_run]
-              result = self.client.add_collab(self.repository, login)
-            end
-            puts "#{login} - #{result}"
-          else
-            puts "#{login} - false (Organization)"
-          end
-        end
-      end
-
-      def run
-        self.promote
-      end
-    end
-  end
-end

data/lib/teachers_pet/commands/fork_collab.rb

@@ -1,12 +0,0 @@
-module TeachersPet
-  class Cli
-    option :repository, required: true, banner: 'OWNER/REPO'
-    option :dry_run, type: :boolean, default: false
-    common_options
-
-    desc "fork_collab", "Give collaborator access to everyone who has forked a particular repository."
-    def fork_collab
-      TeachersPet::Actions::ForkCollab.new(options).run
-    end
-  end
-end