Skip to content

Fix warnings revealed by v0.14.1 PS release #82

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 11 commits into from
May 6, 2021
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,8 @@ New features:
Bugfixes:

Other improvements:
- Fixed warnings revealed by v0.14.1 PS release (#82 by @JordanMartinez)
- Installed transitive dependencies used in source code (#82 by @JordanMartinez)

## [v10.0.0](https://github.com/purescript-contrib/purescript-routing/releases/tag/v10.0.0) - 2021-02-26

Expand Down
11 changes: 11 additions & 0 deletions spago.dhall
Original file line number Diff line number Diff line change
@@ -1,24 +1,35 @@
{ name = "routing"
, dependencies =
[ "aff"
, "arrays"
, "assert"
, "bifunctors"
, "console"
, "control"
, "effect"
, "either"
, "exceptions"
, "foldable-traversable"
, "foreign"
, "integers"
, "js-uri"
, "lists"
, "maybe"
, "newtype"
, "numbers"
, "ordered-collections"
, "partial"
, "prelude"
, "psci-support"
, "record"
, "refs"
, "semirings"
, "strings"
, "transformers"
, "tuples"
, "validation"
, "web-dom"
, "web-events"
, "web-html"
]
, packages = ./packages.dhall
Expand Down
2 changes: 1 addition & 1 deletion src/Routing/Match.purs
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ lit input = Match \route ->
case route of
Cons (Path i) rs | i == input ->
pure $ Tuple rs unit
Cons (Path _) rs ->
Cons (Path _) _ ->
invalid $ free $ UnexpectedPath input
_ ->
invalid $ free ExpectedPathPart
Expand Down
2 changes: 2 additions & 0 deletions test/Test/Browser.purs
Original file line number Diff line number Diff line change
Expand Up @@ -144,6 +144,8 @@ runPushStateTests = withTest \{ assert } -> do
| Rec.equal old' loc5 && Rec.equal new' loc6 -> do
assert "Locations: c/e -> c/e?f" true
hist.pushState (unsafeToForeign 6) "/"
| Rec.equal old' loc6 && Rec.equal new' loc7 -> do
assert "Locations: c/e?f -> /" true
done
_, _ -> do
done
Expand Down