bchess 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 46328051cb5767623ccfc2b65865008bfd3cab4a
-  data.tar.gz: 0d53222053172c2c710a343d19f0738470b8dac0
+  metadata.gz: 370d493302b50f3cd02153f733ab46e43fd27727
+  data.tar.gz: 0436a659709ec9710be92eaa3f54747f6359eda6
 SHA512:
-  metadata.gz: 89624bd2c2b9b8860d3f45cae48264f85010496081a7d8becf9bce98491004912734c582e4f87ee737e4c54ec2de81ce6d8ec8e7464240bb7a6c751ffeeddd95
-  data.tar.gz: 401035c5490e48d066b15871db2e082b4336e93d56df9bcaa523f6de1a1bd9c3985f4a44aa92d3261adcf70460a563121c61424acaa4263011a1c41ba2f26e6d
+  metadata.gz: d824f7dd402d8c83a327f31ab0d63acd88fd0063277897395a402720fd0f23e2def762d0a182747fae6b36a6818fc67806ed8e63738ac55b1967dd106c4c801f
+  data.tar.gz: c3758e3249590c239c537fc36f1a3b526e7089067685cc136bdbb3fa5f612a099299151584fe03f3fe33f910b74c72925e0a8a5df123aeadcff3d6faaf4a6f59

data/Gemfile

@@ -1,6 +1,6 @@
-source "https://rubygems.org"
+source 'https://rubygems.org'
 
-git_source(:github) {|repo_name| "https://github.com/#{repo_name}" }
+git_source(:github) { |repo_name| "https://github.com/#{repo_name}" }
 
 # Specify your gem's dependencies in bchess.gemspec
 gemspec

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    bchess (0.1.1)
+    bchess (0.1.2)
 
 GEM
   remote: https://rubygems.org/

data/Rakefile

@@ -1,6 +1,6 @@
-require "bundler/gem_tasks"
-require "rspec/core/rake_task"
+require 'bundler/gem_tasks'
+require 'rspec/core/rake_task'
 
 RSpec::Core::RakeTask.new(:spec)
 
-task :default => :spec
+task default: :spec

data/bchess.gemspec

@@ -1,29 +1,28 @@
-
-lib = File.expand_path("../lib", __FILE__)
+lib = File.expand_path('lib', __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require "bchess/version"
+require 'bchess/version'
 
 Gem::Specification.new do |spec|
-  spec.name          = "bchess"
+  spec.name          = 'bchess'
   spec.version       = Bchess::VERSION
-  spec.authors       = ["Staszek Zawadzki"]
-  spec.email         = ["s.zawadzki@visuality.pl"]
+  spec.authors       = ['Staszek Zawadzki']
+  spec.email         = ['s.zawadzki@visuality.pl']
 
-  spec.summary       = %q{Parsing chess PGN games.}
-  spec.description   = %q{Parsing and validating a chess games and returning as chess game objects}
-  spec.homepage      = ""
-  spec.license       = "MIT"
+  spec.summary       = 'Parsing chess PGN games.'
+  spec.description   = 'Parsing and validating a chess games and returning as chess game objects'
+  spec.homepage      = 'https://github.com/visualitypl/bchess'
+  spec.license       = 'MIT'
 
   spec.files         = `git ls-files -z`.split("\x0").reject do |f|
     f.match(%r{^(test|spec|features)/})
   end
-  spec.bindir        = "exe"
+  spec.bindir        = 'exe'
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
-  spec.require_paths = ["lib"]
+  spec.require_paths = ['lib']
 
-  spec.add_development_dependency "bundler", "~> 1.16"
-  spec.add_development_dependency "rake", "~> 10.0"
-  spec.add_development_dependency "rspec", "~> 3.0"
-  spec.add_development_dependency "treetop", "~> 1.6"
-  spec.add_development_dependency "pry"
+  spec.add_development_dependency 'bundler', '~> 1.16'
+  spec.add_development_dependency 'pry'
+  spec.add_development_dependency 'rake', '~> 10.0'
+  spec.add_development_dependency 'rspec', '~> 3.0'
+  spec.add_development_dependency 'treetop', '~> 1.6'
 end

data/bin/console

@@ -1,7 +1,7 @@
 #!/usr/bin/env ruby
 
-require "bundler/setup"
-require "bchess"
+require 'bundler/setup'
+require 'bchess'
 
 # You can add fixtures and/or initialization code here to make experimenting
 # with your gem easier. You can also use a different console, if you like.
@@ -10,5 +10,5 @@ require "bchess"
 # require "pry"
 # Pry.start
 
-require "irb"
+require 'irb'
 IRB.start(__FILE__)

data/lib/bchess.rb

@@ -1,33 +1,33 @@
-require "bchess/helpers/board_helpers"
-require "bchess/helpers/fen_helpers"
-require "bchess/helpers/board_helpers"
-require "bchess/helpers/castle_helpers"
-require "bchess/helpers/validations"
-require "bchess/helpers/en_passant_helpers"
-require "bchess/helpers/field_between_helpers"
+require 'bchess/helpers/board_helpers'
+require 'bchess/helpers/fen_helpers'
+require 'bchess/helpers/board_helpers'
+require 'bchess/helpers/castle_helpers'
+require 'bchess/helpers/validations'
+require 'bchess/helpers/en_passant_helpers'
+require 'bchess/helpers/field_between_helpers'
 
-require "bchess/version"
-require "bchess/piece"
-require "bchess/king"
-require "bchess/rook"
-require "bchess/bishop"
-require "bchess/queen"
-require "bchess/knight"
-require "bchess/pawn"
-require "bchess/board"
+require 'bchess/version'
+require 'bchess/piece'
+require 'bchess/king'
+require 'bchess/rook'
+require 'bchess/bishop'
+require 'bchess/queen'
+require 'bchess/knight'
+require 'bchess/pawn'
+require 'bchess/board'
 
-require "pgn/parser"
-require "pgn/pgn_file"
-require "pgn/pgn_nodes"
-require "pgn/game"
-require "pgn/game_header"
-require "pgn/game_body"
-require "pgn/move_info_parser"
+require 'pgn/parser'
+require 'pgn/pgn_file'
+require 'pgn/pgn_nodes'
+require 'pgn/game'
+require 'pgn/game_header'
+require 'pgn/game_body'
+require 'pgn/move_info_parser'
 
 module Bchess
   class InvalidMoveException < StandardError; end
 
-  WHITE = 'w'
-  BLACK = 'b'
-  START_FEN = 'rnbqkbnr/pppppppp/8/8/8/8/PPPPPPPP/RNBQKBNR w KQkq - 0 1'
+  WHITE = 'w'.freeze
+  BLACK = 'b'.freeze
+  START_FEN = 'rnbqkbnr/pppppppp/8/8/8/8/PPPPPPPP/RNBQKBNR w KQkq - 0 1'.freeze
 end

data/lib/bchess/board.rb

@@ -20,6 +20,7 @@ module Bchess
 
     def move(piece, column, row, promoted_piece = nil)
       return false if invalid_data?(piece, column, row)
+
       @en_passant = '-' unless en_passant_detected?(piece, column, row)
 
       if castle_detected?(piece, column)
@@ -30,9 +31,9 @@ module Bchess
         validate_en_passant(piece, column, row) && execute_en_passant(piece, column, row)
       elsif pawn_long_move_detected?(piece, row)
         long_pawn_move(piece, column, row)
-      elsif piece.can_move_to_field?(column,row)
+      elsif piece.can_move_to_field?(column, row)
         piece.move(column, row)
-      elsif piece.can_take_on_field?(column,row)
+      elsif piece.can_take_on_field?(column, row)
         piece.move(column, row)
       else
         return false
@@ -45,7 +46,7 @@ module Bchess
     end
 
     def get_possible_pieces(info)
-      @pieces.select{|p| p.can_make_move?(info, self)}
+      @pieces.select { |p| p.can_make_move?(info, self) }
     end
 
     def update_info(piece, column, row)
@@ -57,7 +58,8 @@ module Bchess
     end
 
     def execute_promotion(piece, column, row, promoted_piece)
-      raise RuntimeError.new("Promotion Not Specified") if promoted_piece.nil? || !(promoted_piece < Bchess::Piece)
+      raise 'Promotion Not Specified' if promoted_piece.nil? || !(promoted_piece < Bchess::Piece)
+
       pieces.delete(piece)
       promoted = promoted_piece.new(piece.color, column, row)
       promoted.moved = true if promoted_piece == Bchess::Rook
@@ -69,11 +71,11 @@ module Bchess
     end
 
     def at(column, row)
-      pieces.select{|p| p.row == row && p.column == column}.first
+      pieces.select { |p| p.row == row && p.column == column }.first
     end
 
     def read_fen
-      fen_pieces, fen_colors, fen_castles, fen_en_passant, fen_halfmove_clock, fen_move_number = fen.strip.split(" ")
+      fen_pieces, fen_colors, fen_castles, fen_en_passant, fen_halfmove_clock, fen_move_number = fen.strip.split(' ')
       set_pieces(fen_pieces)
       set_to_move(fen_colors)
       set_castles(fen_castles)
@@ -84,13 +86,13 @@ module Bchess
 
     def print
       puts 'Pieces WHITE: '
-      puts _pieces(Bchess::WHITE).map{|p| p.to_s }.join(', ')
+      puts _pieces(Bchess::WHITE).map(&:to_s).join(', ')
       puts 'Pieces BLACK: '
-      puts _pieces(Bchess::BLACK).map{|p| p.to_s }.join(', ')
+      puts _pieces(Bchess::BLACK).map(&:to_s).join(', ')
     end
 
     def king(color)
-      @pieces.select{ |p| p.class == Bchess::King && p.color == color }.first
+      @pieces.select { |p| p.class == Bchess::King && p.color == color }.first
     end
 
     def change_to_move
@@ -100,7 +102,7 @@ module Bchess
     private
 
     def remove_old_piece(column, row, color)
-      taken_piece = _other_pieces(color).select{|p| p.row == row && p.column == column}.first
+      taken_piece = _other_pieces(color).select { |p| p.row == row && p.column == column }.first
       pieces.delete(taken_piece)
     end
 
@@ -112,20 +114,20 @@ module Bchess
       _king = king(color)
       _pieces = _other_pieces(color)
 
-      _pieces.any?{|p| attacks?(p, _king)}
+      _pieces.any? { |p| attacks?(p, _king) }
     end
 
     def attacks?(piece, king)
       piece.can_take_on_field?(king.column, king.row) &&
-        piece.fields_between(king.column, king.row).none?{|f| at(*f)}
+        piece.fields_between(king.column, king.row).none? { |f| at(*f) }
     end
 
     def _pieces(color)
-      @pieces.select{ |p| p.color == color }
+      @pieces.select { |p| p.color == color }
     end
 
     def _other_pieces(color)
-      @pieces.select{ |p| p.color != color }
+      @pieces.reject { |p| p.color == color }
     end
   end
 end

data/lib/bchess/game.rb

@@ -2,7 +2,7 @@ module Bchess
   class Game
     attr_reader :fen, :board, :moves
 
-    def initialize(fen='')
+    def initialize(fen = '')
       @fen = fen
       @board = Board.new(fen)
       @moves = []

data/lib/bchess/helpers/board_helpers.rb

@@ -1,27 +1,27 @@
 module Bchess
   module BoardHelpers
     def field(column, row)
-      (column+97).chr + (row+1).to_s
+      (column + 97).chr + (row + 1).to_s
     end
 
     def invalid_data?(piece, column, row)
-      piece.nil? || !(0..7).include?(column) || !(0..7).include?(row)
+      piece.nil? || !(0..7).cover?(column) || !(0..7).cover?(row)
     end
 
     def castle_detected?(piece, column)
-      piece.kind_of?(Bchess::King) && piece.column == 4 && (column - piece.column).abs == 2
+      piece.is_a?(Bchess::King) && piece.column == 4 && (column - piece.column).abs == 2
     end
 
     def promotion_detected?(piece, row)
-      piece.kind_of?(Bchess::Pawn) && (row == 0 || row == 7)
+      piece.is_a?(Bchess::Pawn) && (row == 0 || row == 7)
     end
 
     def en_passant_detected?(piece, column, row)
-      piece.kind_of?(Bchess::Pawn) && piece.row != row && piece.column != column && at(column, row).nil? && piece.can_take_on_field?(column, row)
+      piece.is_a?(Bchess::Pawn) && piece.row != row && piece.column != column && at(column, row).nil? && piece.can_take_on_field?(column, row)
     end
 
     def pawn_long_move_detected?(piece, row)
-      piece.kind_of?(Bchess::Pawn) && (piece.row - row).abs == 2
+      piece.is_a?(Bchess::Pawn) && (piece.row - row).abs == 2
     end
 
     def kings_present?
@@ -37,7 +37,7 @@ module Bchess
     end
 
     def field(column, row)
-      (column+97).chr + (row+1).to_s
+      (column + 97).chr + (row + 1).to_s
     end
   end
 end

data/lib/bchess/helpers/en_passant_helpers.rb

@@ -1,6 +1,6 @@
 module EnPassantHelpers
-  def validate_en_passant(piece, column, row)
-    #TODO
+  def validate_en_passant(_piece, _column, _row)
+    # TODO
     true
   end
 
@@ -17,6 +17,6 @@ module EnPassantHelpers
   def long_pawn_move(piece, column, row)
     piece.move(column, row)
     direction = piece.color == Bchess::WHITE ? 1 : -1
-    @en_passant = field(column, (row + piece.row)/2 - direction)
+    @en_passant = field(column, (row + piece.row) / 2 - direction)
   end
 end

data/lib/bchess/helpers/fen_helpers.rb

@@ -2,25 +2,25 @@ module Bchess
   module FenHelpers
     def fen_hash
       {
-        'k': {klass: Bchess::King, color: Bchess::BLACK},
-        'q': {klass: Bchess::Queen, color: Bchess::BLACK},
-        'r': {klass: Bchess::Rook, color: Bchess::BLACK},
-        'b': {klass: Bchess::Bishop, color: Bchess::BLACK},
-        'n': {klass: Bchess::Knight, color: Bchess::BLACK},
-        'p': {klass: Bchess::Pawn, color: Bchess::BLACK},
-        'K': {klass: Bchess::King, color: Bchess::WHITE},
-        'Q': {klass: Bchess::Queen, color: Bchess::WHITE},
-        'R': {klass: Bchess::Rook, color: Bchess::WHITE},
-        'B': {klass: Bchess::Bishop, color: Bchess::WHITE},
-        'N': {klass: Bchess::Knight, color: Bchess::WHITE},
-        'P': {klass: Bchess::Pawn, color: Bchess::WHITE},
+        'k': { klass: Bchess::King, color: Bchess::BLACK },
+        'q': { klass: Bchess::Queen, color: Bchess::BLACK },
+        'r': { klass: Bchess::Rook, color: Bchess::BLACK },
+        'b': { klass: Bchess::Bishop, color: Bchess::BLACK },
+        'n': { klass: Bchess::Knight, color: Bchess::BLACK },
+        'p': { klass: Bchess::Pawn, color: Bchess::BLACK },
+        'K': { klass: Bchess::King, color: Bchess::WHITE },
+        'Q': { klass: Bchess::Queen, color: Bchess::WHITE },
+        'R': { klass: Bchess::Rook, color: Bchess::WHITE },
+        'B': { klass: Bchess::Bishop, color: Bchess::WHITE },
+        'N': { klass: Bchess::Knight, color: Bchess::WHITE },
+        'P': { klass: Bchess::Pawn, color: Bchess::WHITE }
       }
     end
 
     def write_fen
       result = ''
       7.downto(0) do |i|
-        line_pieces = pieces.select{ |p| p.row == i }
+        line_pieces = pieces.select { |p| p.row == i }
         one_line = create_fen_line(line_pieces)
         result << one_line
         result << '/' unless i == 0
@@ -33,7 +33,7 @@ module Bchess
       counter = 0
 
       0.upto(7) do |i|
-        piece = pieces.select{ |p| p.column == i }.first
+        piece = pieces.select { |p| p.column == i }.first
         if !!piece
           if counter > 0
             line.concat(counter.to_s)
@@ -52,13 +52,13 @@ module Bchess
 
     def set_pieces(board)
       pieces.clear
-      board.split("/").each_with_index do |line, index|
+      board.split('/').each_with_index do |line, index|
         column = 0
         line.each_char do |char|
           if char.to_i != 0
             column += char.to_i - 1
           else
-            pieces << fen_hash[char.to_sym][:klass].new(fen_hash[char.to_sym][:color], column, 7-index)
+            pieces << fen_hash[char.to_sym][:klass].new(fen_hash[char.to_sym][:color], column, 7 - index)
           end
           column += 1
         end
@@ -82,15 +82,15 @@ module Bchess
     end
 
     def to_fen(piece)
-      fen_hash.key({ :klass => piece.class, :color => piece.color }).to_s
+      fen_hash.key(klass: piece.class, color: piece.color).to_s
     end
 
     def change_halfmove_clock(piece)
-      if piece.kind_of?(Bchess::Pawn)
-        @halfmove_clock = 0
-      else
-        @halfmove_clock = halfmove_clock + 1
-      end
+      @halfmove_clock = if piece.is_a?(Bchess::Pawn)
+                          0
+                        else
+                          halfmove_clock + 1
+                        end
     end
 
     def update_castles_after_move(piece)
@@ -104,9 +104,9 @@ module Bchess
 
     def update_castles_after_king_move(color)
       if color == Bchess::WHITE
-        @castles.gsub!('K', '').gsub!('Q', '')
+        @castles.delete!('K').delete!('Q')
       else
-        @castles.gsub!('k', '').gsub!('q', '')
+        @castles.delete!('k').delete!('q')
       end
     end
 

data/lib/bchess/helpers/field_between_helpers.rb

@@ -1,12 +1,12 @@
 module FieldBetweenHelpers
-  def row_fields(dcolumn, drow)
+  def row_fields(dcolumn, _drow)
     smaller, bigger = [column, dcolumn].sort
-    (smaller+1..bigger-1).map{|c| [c, row] }
+    (smaller + 1..bigger - 1).map { |c| [c, row] }
   end
 
-  def column_fields(dcolumn, drow)
+  def column_fields(_dcolumn, drow)
     smaller, bigger = [row, drow].sort
-    (smaller+1..bigger-1).map{|r| [column, r] }
+    (smaller + 1..bigger - 1).map { |r| [column, r] }
   end
 
   def diagonal_fields(dcolumn, drow)
@@ -14,21 +14,21 @@ module FieldBetweenHelpers
     if dcolumn > column
       if drow > row
         (dcolumn - column - 1).times do |i|
-          fields << [dcolumn-(i+1), drow-(i+1)]
+          fields << [dcolumn - (i + 1), drow - (i + 1)]
         end
       else
         (dcolumn - column - 1).times do |i|
-          fields << [dcolumn-(i+1), drow+(i+1)]
+          fields << [dcolumn - (i + 1), drow + (i + 1)]
         end
       end
     else
       if drow > row
         (column - dcolumn - 1).times do |i|
-          fields << [dcolumn+(i+1), drow-(i+1)]
+          fields << [dcolumn + (i + 1), drow - (i + 1)]
         end
       else
         (column - dcolumn - 1).times do |i|
-          fields << [dcolumn+(i+1), drow+(i+1)]
+          fields << [dcolumn + (i + 1), drow + (i + 1)]
         end
       end
     end
@@ -37,6 +37,7 @@ module FieldBetweenHelpers
 
   def fields_between(dcolumn, drow)
     return [] unless can_move_to_field?(dcolumn, drow)
+
     if same_row?(drow)
       row_fields(dcolumn, drow)
     elsif same_column?(dcolumn)

data/lib/bchess/king.rb

@@ -1,6 +1,5 @@
 module Bchess
   class King < Piece
-
     KINGS_REACH = 1
 
     attr_reader :moved
@@ -24,7 +23,7 @@ module Bchess
         (
           by_line(dcolumn, drow, KINGS_REACH) ||
           by_diagonal(dcolumn, drow, KINGS_REACH)
-      )
+        )
     end
   end
 end

data/lib/bchess/knight.rb

@@ -1,6 +1,5 @@
 module Bchess
   class Knight < Piece
-
     def initiialize(*args)
       super(args)
     end
@@ -13,7 +12,7 @@ module Bchess
       super && by_jump(dcolumn, drow)
     end
 
-    def fields_between(column, row)
+    def fields_between(_column, _row)
       []
     end
 

data/lib/bchess/pawn.rb

@@ -1,6 +1,5 @@
 module Bchess
   class Pawn < Piece
-
     def initiialize(*args)
       super(args)
     end
@@ -35,11 +34,11 @@ module Bchess
       column == dcolumn
     end
 
-    def direction_kept?(dcolumn, drow)
+    def direction_kept?(_dcolumn, drow)
       white? ? drow > row : row > drow
     end
 
-    def row_diff?(dcolumn, drow)
+    def row_diff?(_dcolumn, drow)
       (row - drow).abs <= (starting_position? ? 2 : 1)
     end
 
@@ -48,7 +47,7 @@ module Bchess
     end
 
     def pawn_position?
-      (1..6).include?(row)
+      (1..6).cover?(row)
     end
   end
 end

data/lib/bchess/queen.rb

@@ -14,8 +14,8 @@ module Bchess
       super &&
         (
           by_line(dcolumn, drow, QUEEN_REACH) ||
-          by_diagonal(dcolumn, drow,  QUEEN_REACH)
-      )
+          by_diagonal(dcolumn, drow, QUEEN_REACH)
+        )
     end
   end
 end

data/lib/bchess/version.rb

@@ -1,3 +1,3 @@
 module Bchess
-  VERSION = "0.1.1"
+  VERSION = '0.1.2'.freeze
 end

data/lib/pgn/game_header.rb

@@ -8,43 +8,43 @@ module Bchess
       end
 
       def player_white
-        @player_white ||= values["White"]
+        @player_white ||= values['White']
       end
 
       def player_black
-        @player_black ||= values["Black"]
+        @player_black ||= values['Black']
       end
 
       def elo_white
-        @elo_white ||= values["WhiteElo"]
+        @elo_white ||= values['WhiteElo']
       end
 
       def elo_black
-        @elo_black ||= values["BlackElo"]
+        @elo_black ||= values['BlackElo']
       end
 
       def event
-        @event ||= values["Event"]
+        @event ||= values['Event']
       end
 
       def site
-        @site ||= values["Site"]
+        @site ||= values['Site']
       end
 
       def date
-        @date ||= values["Date"]
+        @date ||= values['Date']
       end
 
       def round
-        @round ||= values["Round"]
+        @round ||= values['Round']
       end
 
       def eco
-        @eco ||= values["Eco"]
+        @eco ||= values['Eco']
       end
 
       def result
-        @result ||= values["Result"]
+        @result ||= values['Result']
       end
 
       def values

data/lib/pgn/move_info_parser.rb

@@ -3,11 +3,11 @@ module Bchess
     class MoveInfoParser
       attr_accessor :info, :move, :castle, :move_string
 
-      CHECK = "+"
-      TAKE = "x"
-      TAKE_2 = ":"
-      MATE = "#"
-      PROMOTION = "="
+      CHECK = '+'.freeze
+      TAKE = 'x'.freeze
+      TAKE_2 = ':'.freeze
+      MATE = '#'.freeze
+      PROMOTION = '='.freeze
 
       def initialize(move, castle)
         @move = move
@@ -38,7 +38,7 @@ module Bchess
       end
 
       def extract_piece_type(move_string)
-        if 2 == move_string.size || move_string.size && pawn_taking?(move_string)
+        if move_string.size == 2 || move_string.size && pawn_taking?(move_string)
           Bchess::Pawn
         else
           get_piece_from_letter(move_string[0])
@@ -58,9 +58,9 @@ module Bchess
       end
 
       def extract_additional_info(move_string)
-        if (3 == move_string.size) && pawn_taking?(move_string)
+        if (move_string.size == 3) && pawn_taking?(move_string)
           move_string[0]
-        elsif (4 == move_string.size)
+        elsif move_string.size == 4
           move_string[1]
         end
       end
@@ -79,26 +79,26 @@ module Bchess
       def basic_move_data(move_split)
         @move_string = move_split.last.strip
 
-        raise InvalidMoveException.new("Wrong move description") if move_string.size < 2
+        raise InvalidMoveException, 'Wrong move description' if move_string.size < 2
+
         color = move_split.size == 2 ? Bchess::WHITE : Bchess::BLACK
         number = move_split.first.strip if move_split.size == 2
         {
           piece_color: color,
-          move_number:  number,
-          check:  move_string.include?(CHECK),
+          move_number: number,
+          check: move_string.include?(CHECK),
           take: move_string.include?(TAKE) || move_string.include?(TAKE_2),
-          mate: move_string.include?(MATE),
+          mate: move_string.include?(MATE)
         }
       end
 
-
       def get_piece_from_letter(letter)
         {
-          "K" => Bchess::King,
-          "Q" => Bchess::Queen,
-          "R" => Bchess::Rook,
-          "B" => Bchess::Bishop,
-          "N" => Bchess::Knight
+          'K' => Bchess::King,
+          'Q' => Bchess::Queen,
+          'R' => Bchess::Rook,
+          'B' => Bchess::Bishop,
+          'N' => Bchess::Knight
         }[letter]
       end
     end

data/lib/pgn/parser.rb

@@ -2,7 +2,7 @@ require 'treetop'
 
 module Bchess
   module PGN
-    class ParserException < Exception ;end
+    class ParserException < RuntimeError; end
 
     class Parser
       attr_reader :input, :output, :error, :tree
@@ -15,13 +15,13 @@ module Bchess
       end
 
       def parse
-        if input.kind_of?(String)
+        if input.is_a?(String)
           @tree = @@parser.parse(input)
-        elsif input.kind_of?(Bchess::PGN::PGNFile)
+        elsif input.is_a?(Bchess::PGN::PGNFile)
           @tree = @@parser.parse(input.load_games)
         end
 
-        if !tree
+        unless tree
           raise PGN::ParserException, "Parse error at offset: #{@@parser.index}"
         end
 
@@ -34,8 +34,8 @@ module Bchess
         root_elements = root_node.elements
         return unless root_elements
 
-        root_elements.delete_if{ |node| node.class.name == "Treetop::Runtime::SyntaxNode" }
-        root_elements.each{ |node| sanitize_tree(node) }
+        root_elements.delete_if { |node| node.class.name == 'Treetop::Runtime::SyntaxNode' }
+        root_elements.each { |node| sanitize_tree(node) }
       end
     end
   end

data/lib/pgn/pgn_file.rb

@@ -8,7 +8,7 @@ module Bchess
       end
 
       def load_games
-        file = File.open(filepath, "rt")
+        file = File.open(filepath, 'rt')
         @content = file.read
       end
     end

data/lib/pgn/pgn_nodes.rb

@@ -10,7 +10,7 @@ module Sexp
 
   class PHeader < Treetop::Runtime::SyntaxNode
     def parse(value)
-      val = value.split(" ", 2)
+      val = value.split(' ', 2)
 
       [val[0].delete('['), val[1].delete('"]').rstrip]
     end
@@ -72,11 +72,11 @@ class Treetop::Runtime::SyntaxNode
   def create_value_hash
     hash = {}
 
-    self.elements.each do |element|
+    elements.each do |element|
       key = element.elements.first.text_value
       value = element.elements.last.text_value
 
-      hash[key] = value.gsub('"','')
+      hash[key] = value.delete('"')
     end
 
     hash

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bchess
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Staszek Zawadzki
@@ -24,6 +24,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.16'
+- !ruby/object:Gem::Dependency
+  name: pry
+  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: rake
   requirement: !ruby/object:Gem::Requirement
@@ -66,20 +80,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.6'
-- !ruby/object:Gem::Dependency
-  name: pry
-  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'
 description: Parsing and validating a chess games and returning as chess game objects
 email:
 - s.zawadzki@visuality.pl
@@ -126,7 +126,7 @@ files:
 - lib/pgn/pgn_nodes.rb
 - lib/pgn/pgn_rules.treetop
 - test.sh
-homepage: ''
+homepage: https://github.com/visualitypl/bchess
 licenses:
 - MIT
 metadata: {}