trollolo 0.0.8 → 0.0.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e965a5dd1f21efe913566b48fdd34366c96a281d
-  data.tar.gz: 3ceddeef3b6156896c240274603422f456b4a774
+  metadata.gz: a1b0bc626175dfcdffe5dffa8942ec7469797ec4
+  data.tar.gz: a8feeb3163223a858044f9d2c9be8bb66e350699
 SHA512:
-  metadata.gz: a4a388af4ab07d713d63fcdc1b1ef70601de633e6755a462988893f9cd42cfbf4401fe02a24af1869ef1124f1e09291a711a0f9abb311e0c01e33933b9b07beb
-  data.tar.gz: dc7bd9ccb6dcca2f136163929e68b6842e5cfd1f88da59f5450547a8af4d5a238826b9a1050b880942956b3cc67da2b1789ad568340082b09c3be3932d56b2fd
+  metadata.gz: cf9922a6a5d5ec348be19bde09f391115f6022e1fb0e2a2fc5b5f0c83cae7ee1b3de95037d3964b0fcbb54db1fb16c106a0f85df118f0928aed9f8f1424b7286
+  data.tar.gz: f5345fe994f6db7343aece171c5e57782264f493c454e742aba148f6e58c8d111eb714986ec3f2b2aa99d8bfd3c968b4644f2b16adb67a060b1ece658effa1f7

data/.gitignore

@@ -2,4 +2,3 @@
 Gemfile.lock
 tmp
 coverage
-*.png

data/.travis.yml

@@ -1,11 +1,11 @@
 language: ruby
 cache: bundler
 before_install:
-  - gem update --system
   - gem --version
+  - gem list bundler
 rvm:
-  - "2.1.7"
   - "2.2.3"
+  - "2.3.1"
 script: bundle exec rspec spec/unit
 sudo: false
 

data/CHANGELOG.md

@@ -1,5 +1,20 @@
 # Trollolo Changelog
 
+## Version 0.0.9
+
+* Add `sprint-cleanup` command to move cards back from the sprint board to the
+  planning board. It takes all cards from the "Sprint backlog" and "Doing"
+  columns on the sprint board, moves them to the "Ready" column on the planning
+  board and removes all members and the "under the waterline" label.
+* Add `set-priorities` command to add priorities to the title of all cards of a
+  given column. The priorities are added as a prefix of the form "Pnn", where
+  "nn" is the number of the card in the column. This is useful, if you use the
+  order of cards as priorities and want to move them around to different columns
+  without losing this information.
+* Consistently use hyphens in command names, get rid of underscores.
+* Fix calculation of unplanned tasks on day one
+* Fix scaling tasks
+
 ## Version 0.0.8
 
 * Burndown chart reflects unplanned work

data/Gemfile

@@ -4,9 +4,10 @@ gemspec
 
 group :test do
   gem 'codeclimate-test-reporter'
-  gem 'rspec', '~> 3'
+  gem 'rspec', '~> 3.5'
   gem 'cli_tester'
   gem 'webmock'
+  gem 'vcr'
   gem 'given_filesystem'
   gem 'byebug'
   gem 'awesome_print'

data/README.md

@@ -40,16 +40,20 @@ member_token: 87345897238957a29835789b2374580927f3589072398579820345
 ```
 
 These values have to be set with the personal access data for the Trello API
-and the id of the board, which is processed.
+and the personal access token of the application, which has to be generated.
 
 For creating a developer key go to the
 [Developer API Keys](https://trello.com/1/appKey/generate) page on Trello. It's
 the key in the first box.
 
-For creating a member token use the following URL, replacing `applicationkey` by
+For creating a member token use the following URL, replacing `...` by
 the key you obtained in the first step.
 
-    https://trello.com/1/authorize?key=applicationkey&name=trollolo&expiration=never&response_type=token
+    https://trello.com/1/authorize?key=...&name=trollolo&expiration=never&response_type=token
+
+To create a member token for the `set-priority` command use this URL instead:
+
+    https://trello.com/1/connect?key=...&name=trollolo&response_type=token&scope=read,write
 
 The board id is the cryptic string in the URL of your board.
 

data/lib/card.rb

@@ -18,6 +18,7 @@
 class Card
   # Assuming we have card titles as follows '(8) This is the card name'
   ESTIMATED_REGEX     = /\(([\d.]+)\)/
+  PRIORITY_REGEX      = /^(?:\([\d.]+\) )?P(\d+): /
   SPRINT_NUMBER_REGEX = /\ASprint (\d+)/
 
   def initialize(board_data, card_id)
@@ -42,6 +43,19 @@ class Card
     name.match(ESTIMATED_REGEX).captures.first.to_f
   end
 
+  def priority
+    return unless m = name.match(PRIORITY_REGEX)
+    m.captures.first.to_i
+  end
+
+  def priority=(n)
+    if priority
+      @card_data["name"].sub!(/P\d+: /, "P#{n}: ")
+    else
+      @card_data["name"] = "P#{n}: #{name}"
+    end
+  end
+
   def done_tasks
     count = 0
     @card_data["checklists"].each do |checklist|
@@ -123,4 +137,12 @@ class Card
   def name
     @card_data["name"]
   end
+
+  def name=(str)
+    @card_data["name"] = str
+  end
+
+  def id
+    @card_data["id"]
+  end
 end

data/lib/cli.rb

@@ -206,7 +206,7 @@ EOT
     b.backup(options["board-id"])
   end
 
-  desc "list_backups", "List all backups"
+  desc "list-backups", "List all backups"
   def list_backups
     b = Backup.new @@settings
     b.list.each do |backup|
@@ -214,7 +214,7 @@ EOT
     end
   end
 
-  desc "show_backup", "Show backup of board"
+  desc "show-backup", "Show backup of board"
   option "board-id", :desc => "Id of Trello board", :required => true
   option "show-descriptions", :desc => "Show descriptions of cards", :required => false, :type => :boolean
   def show_backup
@@ -257,7 +257,7 @@ EOT
     trello.set_description(options["card-id"], STDIN.read)
   end
 
-  desc "organization_members", "Show organization members"
+  desc "organization-members", "Show organization members"
   option "org-name", :desc => "Name of organization", :required => true
   def organization_members
     process_global_options options
@@ -293,6 +293,50 @@ EOT
     trello.make_cover(options["card-id"], filename)
   end
 
+  desc "set-priorities", "Set priority text into card titles"
+  long_desc <<EOT
+  Add 'P<n>: ' to the beginning of every cards title, replace where
+  already present. n is the current position of the list on the card.
+EOT
+  option "board-id", :desc => "Id of the board", :required => true
+  option "list-name", :desc => "Name of the list", :required => true
+  def set_priorities
+    process_global_options options
+    require_trello_credentials
+
+    p = Prioritizer.new(@@settings)
+    p.prioritize(options["board-id"], options["list-name"])
+  end
+
+  desc "list-member-boards", "List name and id of all boards"
+  option "member-id", :desc => "Id of the member", :required => true
+  def list_member_boards
+    process_global_options options
+    require_trello_credentials
+
+    trello = TrelloWrapper.new(@@settings)
+    trello.get_member_boards(options["member-id"]).sort_by { |board|
+      board["name"]
+    }.each { |board|
+      puts "#{board["name"]} - #{board["id"]}"
+    }
+  end
+
+  desc "sprint-cleanup", "Move remaining cards to backlog"
+  long_desc <<EOT
+  After the sprint, move remaining cards from 'Sprint Backlog' and 'Doing'
+  back to the planning board into the 'Ready' list.
+EOT
+  option "board-id", :desc => "Id of the board", :required => true
+  option "target-board-id", :desc => "Id of the target board", :required => true
+  def sprint_cleanup
+    process_global_options options
+    require_trello_credentials
+
+    s = SprintCleanup.new(@@settings)
+    s.cleanup(options["board-id"], options["target-board-id"])
+  end
+
   private
 
   def process_global_options options

data/lib/prioritizer.rb

@@ -0,0 +1,34 @@
+class Prioritizer
+
+  def initialize(settings)
+    @settings = settings
+  end
+
+  def prioritize(board_id, list_name)
+    list = find_list(board_id, list_name)
+    fail "list not found on board" unless list
+    update_priorities(list)
+  end
+
+  private
+
+  def trello
+    @trello ||= TrelloWrapper.new(@settings)
+  end
+
+  def update_priorities(list)
+    n = 1
+    list.cards.each do |card|
+      next if card.name =~ /waterline/i
+      card.priority = n
+      trello.set_name(card.id, card.name)
+      puts %(set priority to #{n} for "#{card.name}")
+      n += 1
+    end
+  end
+
+  def find_list(board_id, list_name)
+    board = trello.board(board_id)
+    board.columns.find { |list| list.name == list_name }
+  end
+end

data/lib/sprint_cleanup.rb

@@ -0,0 +1,54 @@
+class SprintCleanup
+  SOURCE_LISTS = ["Sprint Backlog", "Doing"]
+  TARGET_LIST = "Ready"
+
+  def initialize(settings)
+    @settings = settings
+    init_trello
+  end
+
+  def cleanup(board_id, target_board_id)
+    @board = Trello::Board.find(board_id)
+    @target_board = Trello::Board.find(target_board_id)
+
+    SOURCE_LISTS.each do |list_name|
+      move_cards(@board.lists.find { |l| l.name == list_name })
+    end
+  end
+
+  private
+
+  def init_trello
+    Trello.configure do |config|
+      config.developer_public_key = @settings.developer_public_key
+      config.member_token         = @settings.member_token
+    end
+  end
+
+  def target_list
+    @target_list ||= @target_board.lists.find { |l| l.name == TARGET_LIST }
+  end
+
+  def sticky?(card)
+    card.labels.any? { |l| l.name == "Sticky" }
+  end
+
+  def waterline_label(card)
+    card.labels.find { |label| label.name =~ /waterline/i }
+  end
+
+  def remove_waterline_label(card)
+    label = waterline_label(card)
+    card.remove_label(label) if label
+  end
+
+  def move_cards(source_list)
+    source_list.cards.each do |card|
+      next if sticky?(card)
+      puts %(moving card "#{card.name}" to list "#{target_list.name}")
+      card.members.each { |member| card.remove_member(member) }
+      remove_waterline_label(card)
+      card.move_to_board(@target_board, target_list)
+    end
+  end
+end

data/lib/trello_wrapper.rb

@@ -76,10 +76,18 @@ class TrelloWrapper
     card.desc
   end
 
+  def get_member_boards(member_id)
+    JSON.parse(client.get("/members/#{member_id}/boards"))
+  end
+
   def set_description(card_id, description)
     client.put("/cards/#{card_id}/desc?value=#{description}")
   end
 
+  def set_name(card_id, name)
+    client.put("/cards/#{card_id}/name?value=#{name}")
+  end
+
   private
 
   def init_trello

data/lib/trollolo.rb

@@ -27,12 +27,13 @@ require_relative 'settings'
 require_relative 'column'
 require_relative 'card'
 require_relative 'scrum_board'
-require_relative 'result'
 require_relative 'trello_wrapper'
 require_relative 'burndown_chart'
 require_relative 'burndown_data'
 require_relative 'backup'
 require_relative 'checklist'
+require_relative 'prioritizer'
+require_relative 'sprint_cleanup'
 
 class TrolloloError < StandardError
 end

data/lib/version.rb

@@ -1,5 +1,5 @@
 module Trollolo
 
-  VERSION = "0.0.8"
+  VERSION = "0.0.9"
 
 end

data/scripts/burndowndata.py

@@ -9,14 +9,12 @@ class BurndownData:
     self.args = args
     burndown = self.readYAML(self.args.sprint)
     self.getSprintData(burndown)
-    self.calculateMaxStoryPoints()
-    self.calculateMaxTasks()
     self.setBonusTasksDayOne(burndown)
     self.setUnplannedTasksDayOne(burndown)
     self.setExtraDays()
     self.setUnplannedDays()
-    self.calculateYRange(self.max_story_points, self.bonus_tasks_done, self.bonus_story_points_done, self.unplanned_tasks_done, self.unplanned_story_points_done)
-    self.setScaleFactor(self.max_tasks, self.max_story_points)
+    self.calculateYRange(self.total_story_points[0], self.bonus_tasks_done, self.bonus_story_points_done, self.unplanned_tasks_done, self.unplanned_story_points_done)
+    self.setScaleFactor(self.total_tasks[0], self.total_story_points[0])
 
   def readYAML(self, sprint_number):
     with open('burndown-data-' + sprint_number + '.yaml', 'r') as f:
@@ -49,8 +47,6 @@ class BurndownData:
     self.y_fast_lane = []
     self.total_fast_lane = []
     self.total_unplanned_fast_lane = []
-    self.max_story_points = 0
-    self.max_tasks = 0
 
     for day in burndown["days"]:
       self.days.append(self.current_day)
@@ -94,16 +90,6 @@ class BurndownData:
       self.current_day += 1
     return
 
-  def calculateMaxStoryPoints(self):
-    for sp in self.total_story_points:
-      self.max_story_points = max(self.max_story_points, sp)
-    return
-
-  def calculateMaxTasks(self):
-    for t in self.total_tasks:
-      self.max_tasks = max(self.max_tasks, t)
-    return
-
   def setBonusTasksDayOne(self, burndown):
     if burndown["days"][0].has_key("tasks_extra"):
       self.bonus_tasks_day_one = burndown["days"][0]["tasks_extra"]["done"]
@@ -113,7 +99,7 @@ class BurndownData:
 
   def setUnplannedTasksDayOne(self, burndown):
     if burndown["days"][0].has_key("unplanned_tasks"):
-      self.unplanned_tasks_day_one = burndown["days"][0]["unplanned_tasks"]["done"]
+      self.unplanned_tasks_day_one = burndown["days"][0]["unplanned_tasks"]["total"] - burndown["days"][0]["unplanned_tasks"]["open"]
     else:
       self.unplanned_tasks_day_one = 0
     return
@@ -140,8 +126,8 @@ class BurndownData:
       self.unplanned_day = 1
     return
 
-  def calculateYRange(self, max_story_points, bonus_tasks_done, bonus_story_points_done, unplanned_tasks_done, unplanned_story_points_done):
-    self.ymax = max_story_points + 3
+  def calculateYRange(self, total_story_points, bonus_tasks_done, bonus_story_points_done, unplanned_tasks_done, unplanned_story_points_done):
+    self.ymax = total_story_points + 3
 
     if len(bonus_tasks_done) > 0:
       ymin_bonus_tasks = min(bonus_tasks_done) -3
@@ -168,7 +154,7 @@ class BurndownData:
       self.ymin = -3
     return
 
-  def setScaleFactor(self, max_tasks, max_story_points):
-    self.scalefactor = float(max_tasks) / float(max_story_points)
+  def setScaleFactor(self, total_tasks, total_story_points):
+    self.scalefactor = float(total_tasks) / float(total_story_points)
     return
 

data/spec/data/create_burndown_helper/burndown-data-01.yaml

@@ -0,0 +1,23 @@
+---
+meta:
+  board_id: CRdddpdy
+  sprint: 1
+  total_days: 10
+  weekend_lines:
+  - 3.5
+  - 8.5
+days:
+- date: '2016-03-17'
+  updated_at: '2016-03-17T18:31:46+01:00'
+  story_points:
+    total: 16.0
+    open: 13.0
+  tasks:
+    total: 13
+    open: 9
+  unplanned_story_points:
+    total: 3.0
+    open: 1.0
+  unplanned_tasks:
+    total: 2
+    open: 1