Apply suggestions from code review

Co-authored-by: Nick Rolfe <[email protected]>

Author: aibaars

ruby/ql/lib/codeql/ruby/InclusionTests.qll

@@ -38,7 +38,7 @@ class InclusionTest extends DataFlow::Node instanceof InclusionTest::Range {
  */
 module InclusionTest {
   /**
-   * A expression that is equivalent to `A.include?(B)` or `!A.include?(B)`.
+   * An expression that is equivalent to `A.include?(B)` or `!A.include?(B)`.
    *
    * Note that this also includes calls to the array method named `include?`.
    */

ruby/ql/lib/codeql/ruby/StringOps.qll

@@ -104,7 +104,7 @@ module StringOps {
   }
 
   /**
-   * A expression that is equivalent to `A.include?(B)` or `!A.include?(B)`.
+   * An expression that is equivalent to `A.include?(B)` or `!A.include?(B)`.
    * Note that this class is equivalent to `InclusionTest`, which also matches
    * inclusion tests on array objects.
    */

ruby/ql/src/queries/security/cwe-020/examples/IncompleteUrlSubstringSanitization_BAD1.rb

@@ -4,5 +4,6 @@ def index
         # BAD: the host of `url` may be controlled by an attacker
         if url.include?("example.com")
             redirect_to url
+        end
     end
 end