Added GitHub workflow

.github/workflows/gradle.yml

@@ -0,0 +1,61 @@
+name: gradle-ci
+
+on: [push, pull_request, workflow_dispatch]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+
+    env:
+      GRADLE_OPTS: "-Dorg.gradle.jvmargs=-XX:MaxMetaspaceSize=512m"
+      SONAR_JDK: "11"
+
+    strategy:
+      matrix:
+        java-version: [ 11, 17, 19 ]
+
+    steps:
+      - uses: actions/checkout@v2
+        with:
+          fetch-depth: 0
+
+      - name: Set up JDK ${{ matrix.java-version }}
+        uses: actions/setup-java@v1
+        with:
+          java-version: ${{ matrix.java-version }}
+
+      - name: Grant execute permission for gradlew
+        run: chmod +x gradlew
+
+      - name: Cache SonarCloud packages
+        if: matrix.java-version == env.SONAR_JDK
+        uses: actions/cache@v1
+        with:
+          path: ~/.sonar/cache
+          key: ${{ runner.os }}-sonar
+          restore-keys: ${{ runner.os }}-sonar
+
+      - name: Cache Gradle packages
+        uses: actions/cache@v2
+        with:
+          path: |
+            ~/.gradle/caches
+            ~/.gradle/wrapper
+          key: ${{ runner.os }}-gradle-${{ matrix.java-version }}-${{ hashFiles('**/*.gradle*', '**/gradle-wrapper.properties') }}
+          restore-keys: |
+            ${{ runner.os }}-gradle-${{ matrix.java-version }}-
+
+      - name: Test with Gradle
+        run: ./gradlew build check --stacktrace
+
+      - name: SonarCloud
+        if: success() && matrix.java-version == env.SONAR_JDK
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
+        run: ./gradlew sonarqube
+
+      - name: Cleanup Gradle Cache
+        run: |
+          rm -f ~/.gradle/caches/modules-2/modules-2.lock
+          rm -f ~/.gradle/caches/modules-2/gc.properties

lib/build.gradle.kts

@@ -16,12 +16,12 @@ plugins {
     id("signing")
 }
 
-description = "URL parameters encoding and decoding"
+description = "Encode and decode URL parameters"
 group = "net.thauvin.erik"
 version = "0.9-SNAPSHOT"
 
 val deployDir = "deploy"
-val gitHub = "ethauvin/$name"
+val gitHub = "ethauvin/${rootProject.name}"
 val mavenUrl = "https://github.com/$gitHub"
 val publicationName = "mavenJava"
 
@@ -42,7 +42,8 @@ java {
 
 sonarqube {
     properties {
-        property("sonar.projectKey", "ethauvin_$name")
+        property("sonar.projectName", rootProject.name)
+        property("sonar.projectKey", "ethauvin_${rootProject.name}")
         property("sonar.organization", "ethauvin-github")
         property("sonar.host.url", "https://sonarcloud.io")
         property("sonar.sourceEncoding", "UTF-8")
@@ -142,8 +143,8 @@ publishing {
                     }
                 }
                 scm {
-                    connection.set("scm:git:git://github.com/$gitHub.git")
-                    developerConnection.set("scm:git:git@github.com:$gitHub.git")
+                    connection.set("scm:git://github.com/$gitHub.git")
+                    developerConnection.set("scm:git@github.com:$gitHub.git")
                     url.set(mavenUrl)
                 }
                 issueManagement {

lib/pom.xml

@@ -1,51 +1,53 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<project xmlns="http://maven.apache.org/POM/4.0.0" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
-  <!-- This module was also published with a richer model, Gradle metadata,  -->
-  <!-- which should be used instead. Do not delete the following line which  -->
-  <!-- is to indicate to Gradle or any Gradle module metadata file consumer  -->
-  <!-- that they should prefer consuming it instead. -->
-  <!-- do_not_remove: published-with-gradle-metadata -->
-  <modelVersion>4.0.0</modelVersion>
-  <groupId>net.thauvin.erik</groupId>
-  <artifactId>urlencoder</artifactId>
-  <version>0.9-SNAPSHOT</version>
-  <name>urlencoder</name>
-  <description>URL parameters encoding and decoding</description>
-  <url>https://github.com/ethauvin/lib</url>
-  <licenses>
-    <license>
-      <name>The Apache License, Version 2.0</name>
-      <url>http://www.apache.org/licenses/LICENSE-2.0.txt</url>
-    </license>
-  </licenses>
-  <developers>
-    <developer>
-      <id>gbevin</id>
-      <name>Geert Bevin</name>
-      <email>gbevin@uwyn.com</email>
-    </developer>
-    <developer>
-      <id>ethauvin</id>
-      <name>Erik C. Thauvin</name>
-      <email>erik@thauvin.net</email>
-      <url>https://erik.thauvin.net/</url>
-    </developer>
-  </developers>
-  <scm>
-    <connection>scm:git:git://github.com/ethauvin/lib.git</connection>
-    <developerConnection>scm:git:git@github.com:ethauvin/lib.git</developerConnection>
-    <url>https://github.com/ethauvin/lib</url>
-  </scm>
-  <issueManagement>
-    <system>GitHub</system>
-    <url>https://github.com/ethauvin/lib/issues</url>
-  </issueManagement>
-  <dependencies>
-    <dependency>
-      <groupId>org.jetbrains.kotlin</groupId>
-      <artifactId>kotlin-stdlib-jdk8</artifactId>
-      <version>1.8.0</version>
-      <scope>compile</scope>
-    </dependency>
-  </dependencies>
+<project xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xmlns="http://maven.apache.org/POM/4.0.0"
+         xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd">
+    <!-- This module was also published with a richer model, Gradle metadata,  -->
+    <!-- which should be used instead. Do not delete the following line which  -->
+    <!-- is to indicate to Gradle or any Gradle module metadata file consumer  -->
+    <!-- that they should prefer consuming it instead. -->
+    <!-- do_not_remove: published-with-gradle-metadata -->
+    <modelVersion>4.0.0</modelVersion>
+    <groupId>net.thauvin.erik</groupId>
+    <artifactId>urlencoder</artifactId>
+    <version>0.9-SNAPSHOT</version>
+    <name>urlencoder</name>
+    <description>Encode and decode URL parameters</description>
+    <url>https://github.com/ethauvin/urlencoder</url>
+    <licenses>
+        <license>
+            <name>The Apache License, Version 2.0</name>
+            <url>http://www.apache.org/licenses/LICENSE-2.0.txt</url>
+        </license>
+    </licenses>
+    <developers>
+        <developer>
+            <id>gbevin</id>
+            <name>Geert Bevin</name>
+            <email>gbevin@uwyn.com</email>
+        </developer>
+        <developer>
+            <id>ethauvin</id>
+            <name>Erik C. Thauvin</name>
+            <email>erik@thauvin.net</email>
+            <url>https://erik.thauvin.net/</url>
+        </developer>
+    </developers>
+    <scm>
+        <connection>scm:git://github.com/ethauvin/urlencoder.git</connection>
+        <developerConnection>scm:git@github.com:ethauvin/urlencoder.git</developerConnection>
+        <url>https://github.com/ethauvin/urlencoder</url>
+    </scm>
+    <issueManagement>
+        <system>GitHub</system>
+        <url>https://github.com/ethauvin/urlencoder/issues</url>
+    </issueManagement>
+    <dependencies>
+        <dependency>
+            <groupId>org.jetbrains.kotlin</groupId>
+            <artifactId>kotlin-stdlib-jdk8</artifactId>
+            <version>1.8.0</version>
+            <scope>compile</scope>
+        </dependency>
+    </dependencies>
 </project>

lib/src/main/kotlin/net/thauvin/erik/urlencoder/UrlEncoder.kt

@@ -50,7 +50,7 @@ object UrlEncoder {
 
     // see https://www.rfc-editor.org/rfc/rfc3986#page-13
     private fun Char.isUnreserved(): Boolean {
-        return if (this > '~') false else unreservedChars.get(this.code)
+        return this <= '~' && unreservedChars.get(code)
     }
 
     private fun StringBuilder.appendEncodedDigit(digit: Int) {
@@ -122,7 +122,7 @@ object UrlEncoder {
 
     /**
      * Transforms a provided [String] object into a new string, containing only valid URL characters in the UTF-8
-     * encoding. Letters, numbers, unreserved (<code>"_-!.~'()*"</code>) and allowed characters are left intact.
+     * encoding. Letters, numbers, unreserved (`_-!.~'()*`) and allowed characters are left intact.
      */
     @JvmStatic
     fun encode(source: String, vararg allow: Char): String {