A little code cleanup. (#61)

* A little code cleanup. Make sure things all run on latest stuff. Adding windows to the CI

* update ameba with latest suggestions

.github/workflows/ci.yml

@@ -32,12 +32,17 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
+        os: [ubuntu-latest]
         crystal_version:
-          - 1.4.0
+          - 1.6.2
           - latest
         experimental:
           - false
-    runs-on: ubuntu-latest
+        include:
+          - os: windows-latest
+            crystal_version: latest
+            experimental: true
+    runs-on: ${{ matrix.os }}
     continue-on-error: ${{ matrix.experimental }}
     steps:
       - uses: actions/checkout@v2

shard.yml

@@ -10,4 +10,4 @@ license: MIT
 development_dependencies:
   ameba:
     github: crystal-ameba/ameba
-    version: ~> 1.0.0
+    version: ~> 1.4.0

src/lucky_router/fragment.cr

@@ -81,7 +81,7 @@ class LuckyRouter::Fragment(T)
     end
   end
 
-  def dynamic?
+  def dynamic? : Bool
     path_part.path_variable?
   end
 

src/lucky_router/path_part.cr

@@ -37,20 +37,20 @@ struct LuckyRouter::PathPart
   def initialize(@part)
   end
 
-  def name
+  def name : String
     name = part.lchop('?').lchop('*').lchop(':')
     unnamed_glob?(name) ? "glob" : name
   end
 
-  def optional?
+  def optional? : Bool
     part.starts_with?('?')
   end
 
-  def path_variable?
+  def path_variable? : Bool
     part.starts_with?(':') || part.starts_with?("?:") || glob?
   end
 
-  def glob?
+  def glob? : Bool
     part.starts_with?('*')
   end
 

src/lucky_router/path_reader.cr

@@ -23,7 +23,7 @@ struct LuckerRouter::PathReader
   def initialize(@path : String)
   end
 
-  def each
+  def each(&)
     each_segment do |offset, length, decode|
       segment = String.new(@path.to_unsafe + offset, length)
       if decode
@@ -34,7 +34,7 @@ struct LuckerRouter::PathReader
     end
   end
 
-  private def each_segment
+  private def each_segment(&)
     index = 0
     offset = 0
     decode = false