Merge pull request #1643 from wonda-tea-coffee/fix-typo

fix typo

Author: presidentbeef

CHANGES.md

@@ -449,7 +449,7 @@
 * Delay loading vendored gems and modifying load path
 * Avoid warning about SQL injection with `quoted_primary_key`
 * Support more safe `&.` operations
-* Allow multile line regex in `validates_format_of` (Dmitrij Fedorenko)
+* Allow multiple line regex in `validates_format_of` (Dmitrij Fedorenko)
 * Only consider `if` branches in templates
 * Avoid overwriting instance/class methods with same name (Tim Wade)
 * Add `--force-scan` option (Neil Matatall)

lib/brakeman.rb

@@ -394,7 +394,7 @@ def self.scan options
     if options[:parallel_checks]
       notify "Running checks in parallel..."
     else
-      notify "Runnning checks..."
+      notify "Running checks..."
     end
 
     tracker.run_checks
@@ -479,7 +479,7 @@ def self.debug message
     $stderr.puts message if @debug
   end
 
-  # Compare JSON ouptut from a previous scan and return the diff of the two scans
+  # Compare JSON output from a previous scan and return the diff of the two scans
   def self.compare options
     require 'json'
     require 'brakeman/differ'

lib/brakeman/app_tree.rb

@@ -28,7 +28,7 @@ def self.from_options(options)
     # Accepts an array of filenames and paths with the following format and
     # returns a Regexp to match them:
     #   * "path1/file1.rb" - Matches a specific filename in the project directory.
-    #   * "path1/" - Matches any path that conatains "path1" in the project directory.
+    #   * "path1/" - Matches any path that contains "path1" in the project directory.
     #   * "/path1/ - Matches any path that is rooted at "path1" in the project directory.
     #
     def self.regex_for_paths(paths)

lib/brakeman/checks/check_json_parsing.rb

@@ -74,7 +74,7 @@ def check_json_version name, version
     warning_type = "Denial of Service"
     confidence = :medium
     gem_name = "#{name} gem"
-    message = msg(msg_version(version, gem_name), " has a symbol creation vulnerablity. Upgrade to ")
+    message = msg(msg_version(version, gem_name), " has a symbol creation vulnerability. Upgrade to ")
 
     if version >= "1.7.0"
       confidence = :high

lib/brakeman/report/report_sarif.rb

@@ -93,7 +93,7 @@ def results
     end
   end
 
-  # Returns a hash of all check descriptions, keyed by check namne
+  # Returns a hash of all check descriptions, keyed by check name
   def check_descriptions
     @check_descriptions ||= Brakeman::Checks.checks.map do |check|
       [check.name.gsub(/^Check/, ''), check.description]

lib/ruby_parser/bm_sexp.rb

@@ -544,7 +544,7 @@ def body_list
   end
 
   # Number of "statements" in a method.
-  # This is more effecient than `Sexp#body.length`
+  # This is more efficient than `Sexp#body.length`
   # because `Sexp#body` creates a new Sexp.
   def method_length
     expect :defn, :defs

test/apps/rails2/public/javascripts/prototype.js

@@ -4162,7 +4162,7 @@ var $continue = new Error('"throw $continue" is deprecated, use "return" instead
 // further below, that map to the newer Element methods.
 var Position = {
   // set to true if needed, warning: firefox performance problems
-  // NOT neeeded for page scrolling, only if draggable contained in
+  // NOT needed for page scrolling, only if draggable contained in
   // scrollable elements
   includeScrollOffsets: false,
 

test/apps/rails_with_xss_plugin/public/javascripts/prototype.js

@@ -4162,7 +4162,7 @@ var $continue = new Error('"throw $continue" is deprecated, use "return" instead
 // further below, that map to the newer Element methods.
 var Position = {
   // set to true if needed, warning: firefox performance problems
-  // NOT neeeded for page scrolling, only if draggable contained in
+  // NOT needed for page scrolling, only if draggable contained in
   // scrollable elements
   includeScrollOffsets: false,