.github/workflows/test.yml

@@ -1,10 +1,7 @@
 name: Run Tests
 
 on:
-  pull_request:
+  [push, pull_request]
-  push:
-    branches:
-      - '**'
 
 env:
   nim-version: 'stable'

nulid.nimble

@@ -1,6 +1,6 @@
 # Package
 
-version       = "1.3.0"
+version       = "1.2.0"
 author        = "Yu Vitaqua fer Chronos"
 description   = "An implementation of ULID!"
 license       = "CC0"

src/nulid.nim

@@ -1,6 +1,5 @@
 import std/[
-  times,
+  times
-  json
 ]
 
 import crockfordb32
@@ -29,9 +28,10 @@ when not NoLocks:
 Note: There are 2 defines that can be passed to the compiler to trigger different
 functionality in this library at runtime, they are listed here:
   - `--define:nulidInsecureRandom`: Uses `std/random` instead of `std/sysrand`.
-  - `--define:nulidNoLocks`: Disables the use of locks.
+  - `--define:nulidNoLocks`
 
-The JS backend used `-d:nulidNoLocks` by default.
+The JS backend used `-d:nulidNoLocks` by default and Nimscript uses both.
+these flags by default (whether either work with NULID is untested).
 ]##
 
 when not defined(js):
@@ -176,9 +176,6 @@ proc wait(gen: ULIDGenerator): int64 {.gcsafe.} =
 proc ulid*(gen: ULIDGenerator, timestamp = LowInt48, randomness = LowUint80): ULID {.gcsafe.} =
   ## Generate a `ULID`, if timestamp is equal to `0`, the `randomness` parameter
   ## will be ignored.
-  ##
-  ## See also:
-  ## * `ulid(int64, UInt128) <#ulid_2>`_
   runnableExamples:
     let gen = initUlidGenerator()
 
@@ -327,14 +324,6 @@ func `$`*(ulid: ULID): string =
   else:
     result = JsBigInt.encode(ulid.toInt128(), 26)
 
-# std/json support
-proc `%`*(u: ULID): JsonNode = newJString($u)
-
-proc to*(j: JsonNode, _: typedesc[ULID]): ULID =
-  if j.kind != JString:
-    raise newException(JsonKindError, "Expected a string!")
-
-  result = ULID.parse(j.getStr())
 
 when HasJsony:
   import jsony

tests/test1.nim

@@ -4,10 +4,8 @@
 # the letter 't').
 #
 # To run these tests, simply execute `nimble test`.
-import std/[
+
-  unittest,
+import unittest
-  json
-]
 
 const UlidRandStr = "541019288874337045949482"
 
@@ -49,9 +47,3 @@ when not defined(js):
 
     check ulid == ULID.fromBytes(ulidBytes)
     check ulid.toBytes == ulidBytes
-
-test "ULID std/json support":
-  let ulid = ULID.parse("01H999MBGTEA8BDS0M5AWEBB1A")
-
-  check (%ulid).getStr() == "01H999MBGTEA8BDS0M5AWEBB1A"
-  check (%ulid).to(ULID) == ulid