sapristi-tool/sapristi

View on GitHub

Showing 114 of 114 total issues

Line is too long. [99/80]
Open

        raise OptionParser::InvalidOption, "requires a wait time <= 120 seconds, provided=#{value}"
Severity: Minor
Found in lib/sapristi/arguments_parser.rb by rubocop

Line is too long. [111/80]
Open

      new_windows.each { |window| ::Sapristi.logger.debug "  Found new window=#{window.pid}: #{window.title}" }

Line is too long. [92/80]
Open

      raise Error, "invalid workspace=#{id} valid=#{available}" unless available.include? id
Severity: Minor
Found in lib/sapristi/window_manager.rb by rubocop

Line is too long. [112/80]
Open

      raise Error, "window y dimensions: [#{y_pos}, #{y_end}] exceeds monitor height [0..#{monitor_height - 1}]"
Severity: Minor
Found in lib/sapristi/definition_parser.rb by rubocop

Line is too long. [85/80]
Open

    # spec.metadata["source_code_uri"] = "TODO: Put your gem's public repo URL here."
Severity: Minor
Found in sapristi.gemspec by rubocop

Line is too long. [87/80]
Open

  # The `git ls-files -z` loads the files in the RubyGem that have been added into git.
Severity: Minor
Found in sapristi.gemspec by rubocop

Line is too long. [105/80]
Open

        ::Sapristi.logger.info "Launch #{cmd.split[0]}, process=#{process_pid} pgroup=#{Process.getpgrp}"

Line is too long. [120/80]
Open

        log.debug "Found window by process group=#{Process.getpgid(window.pid)}, id=#{window.id}, title=#{window.title}"

Line is too long. [82/80]
Open

      @monitor = MonitorManager.new.get_monitor_or_main definition_hash['Monitor']
Severity: Minor
Found in lib/sapristi/definition.rb by rubocop

Line is too long. [94/80]
Open

                     .each_with_object({}) { |monitor, memo| memo[monitor['name']] = monitor }

Do not freeze immutable objects, as freezing them has no effect.
Open

      MONITOR_LINE_REGEX = /^\s*+(?<id>[0-9]+):\s*\+(?<main>\*)?(?<name>[^\s]+)\s+#{RESOLUTION}\+#{OFFSET}.*$/.freeze

This cop check for uses of Object#freeze on immutable objects.

Example:

# bad
CONST = 1.freeze

# good
CONST = 1

Line is too long. [83/80]
Open

      geometry_field_nil = %w[Width Height X Y].find { |key| definition[key].nil? }
Severity: Minor
Found in lib/sapristi/definition.rb by rubocop

Line is too long. [102/80]
Open

        raise Error, "Unable to process configuration file: #{file}, line=#{line}, error=#{e.message}"

Line is too long. [102/80]
Open

    def initialize(window_manager = WindowManager.new, process_manager = NewProcessWindowDetector.new)

Line is too long. [102/80]
Open

        ::Sapristi.logger.info "Found existing window pid=#{windows[0].pid} title=#{windows[0].title}"

Line is too long. [84/80]
Open

    # spec.metadata["changelog_uri"] = "TODO: Put your gem's CHANGELOG.md URL here."
Severity: Minor
Found in sapristi.gemspec by rubocop

Line is too long. [103/80]
Open

        remove_extended_hints(window) if window.maximized_horizontally? || window.maximized_vertically?

Line is too long. [172/80]
Open

        log.warn "We can not be sure window '#{window.title}' with pid=#{window.pid} is related to program=#{program}, pid=#{waiter.pid}, status=#{waiter.status || 'dead'}"

Line is too long. [105/80]
Open

    ATTRIBUTES = %i[id main name x y offset_x offset_y work_area work_area_width work_area_height].freeze
Severity: Minor
Found in lib/sapristi/monitor.rb by rubocop

Line is too long. [88/80]
Open

      raise Error, "Invalid configuration file: Empty file #{csv_file}" if table.eql? []
Severity
Category
Status
Source
Language