Skip to content
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

Applicative parser-printer #61

Merged
merged 41 commits into from Oct 19, 2017
Merged
Changes from all commits
Commits
Show all changes
41 commits
Select commit Hold shift + click to select a range
fb231b4
Added a bunch of tests for router.
mbrandonw Oct 7, 2017
6ebbd82
small clean up
mbrandonw Oct 12, 2017
482c4bb
dont need no stinkin monad
mbrandonw Oct 12, 2017
60665e8
make another route
mbrandonw Oct 13, 2017
7a71fbf
array of routes
mbrandonw Oct 13, 2017
46cdc1c
wip
mbrandonw Oct 13, 2017
b403a10
wtf crasher
mbrandonw Oct 13, 2017
34ea28f
stuffff
mbrandonw Oct 14, 2017
d651065
clean up
mbrandonw Oct 14, 2017
dd7cba8
clean up
mbrandonw Oct 14, 2017
1dcd38f
curry\!
mbrandonw Oct 14, 2017
3b1f28c
curry
mbrandonw Oct 14, 2017
6e49364
stuff
mbrandonw Oct 14, 2017
e683c88
basic codable stuff in params
mbrandonw Oct 14, 2017
5e541d6
better name for curry
mbrandonw Oct 14, 2017
793886a
clean up
mbrandonw Oct 14, 2017
e440bb2
lots of clean up
mbrandonw Oct 15, 2017
41b3955
more clean up
mbrandonw Oct 15, 2017
ac0c0f2
clean
mbrandonw Oct 15, 2017
77640eb
move stuff
mbrandonw Oct 15, 2017
12e15cb
more clean
mbrandonw Oct 15, 2017
4b36a82
clean
mbrandonw Oct 15, 2017
1d91615
getting real tests in
mbrandonw Oct 16, 2017
4a81e61
clean up
mbrandonw Oct 16, 2017
33371bb
todo
mbrandonw Oct 16, 2017
36a9d24
clean up
mbrandonw Oct 17, 2017
bf720f5
removed old parser
mbrandonw Oct 17, 2017
d1d817f
clean up
mbrandonw Oct 17, 2017
512ba18
clean up
mbrandonw Oct 17, 2017
76bf33a
consume get
mbrandonw Oct 18, 2017
7d9f531
more tests
mbrandonw Oct 18, 2017
859c337
Revert "more tests"
mbrandonw Oct 18, 2017
b1f7793
Revert "Revert "more tests""
mbrandonw Oct 18, 2017
bb58367
fix test
mbrandonw Oct 18, 2017
7502ac3
rename image/preimage to apply/unapply
mbrandonw Oct 19, 2017
91950b3
Remove TODO
stephencelis Oct 19, 2017
bdae769
_end -> end
stephencelis Oct 19, 2017
80aae9c
Play around with simpler boilerplate
stephencelis Oct 19, 2017
947f57f
pr feedback
mbrandonw Oct 19, 2017
052d797
Merge remote-tracking branch 'origin/master' into parser-printer
mbrandonw Oct 19, 2017
244a4eb
Merge branch 'parser-printer' of https://github.com/pointfreeco/swift…
mbrandonw Oct 19, 2017
File filter...
Filter file types
Jump to…
Jump to file
Failed to load files.

Always

Just for now

@@ -50,17 +50,17 @@
"package": "Prelude",
"repositoryURL": "https://github.com/pointfreeco/swift-prelude.git",
"state": {
"branch": "aab186e",
"revision": "aab186e757d88efbca2f17939dd7b94731744852",
"branch": "a8a4849",
"revision": "a8a48493685da8067b7605ce1094e42e9993598a",
"version": null
}
},
{
"package": "SnapshotTesting",
"repositoryURL": "https://github.com/pointfreeco/swift-snapshot-testing.git",
"state": {
"branch": "6a292e9",
"revision": "6a292e9fe1d994207f1bf47fefdb78ca44951a09",
"branch": "b4081b7",
"revision": "b4081b7f14eac7d0540b6cf6a4caf57816d7a8c2",
"version": null
}
},
@@ -21,14 +21,14 @@ let package = Package(
.library(name: "MediaType", targets: ["MediaType"]),
],
dependencies: [
.package(url: "https://github.com/pointfreeco/swift-prelude.git", .revision("aab186e")),
.package(url: "https://github.com/pointfreeco/swift-snapshot-testing.git", .revision("6a292e9")),
.package(url: "https://github.com/pointfreeco/swift-prelude.git", .revision("a8a4849")),
.package(url: "https://github.com/pointfreeco/swift-snapshot-testing.git", .revision("b4081b7")),
.package(url: "https://github.com/bkase/DoctorPretty.git", .exact("0.3.0")),
.package(url: "https://github.com/IBM-Swift/BlueCryptor.git", .exact("0.8.17"))
],
targets: [
.target(name: "ApplicativeRouter", dependencies: ["Either", "Prelude"]),
.testTarget(name: "ApplicativeRouterTests", dependencies: ["ApplicativeRouter", "Optics"]),
.target(name: "ApplicativeRouter", dependencies: ["Either", "Optics", "Prelude"]),

This comment has been minimized.

@mbrandonw

mbrandonw Oct 18, 2017
Author Member

I could do without Optics in the router if we didn't wanna bring in this dependency. just hard to say no to optics!

.testTarget(name: "ApplicativeRouterTests", dependencies: ["ApplicativeRouter", "Deriving", "Optics"]),

.target(name: "ApplicativeRouterHttpPipelineSupport",
dependencies: ["ApplicativeRouter", "HttpPipeline", "Prelude"]),

This file was deleted.

ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.