This page is not created by, affiliated with, or supported by Slack Technologies, Inc.
2022-04-10
Channels
- # announcements (16)
- # asami (7)
- # aws (5)
- # beginners (46)
- # calva (47)
- # clj-kondo (3)
- # cljdoc (23)
- # clojure (70)
- # clojure-germany (1)
- # clojurescript (6)
- # community-development (2)
- # datomic (3)
- # emacs (14)
- # exercism (5)
- # hyperfiddle (4)
- # jobs (1)
- # kaocha (9)
- # lsp (13)
- # malli (6)
- # membrane (11)
- # off-topic (22)
- # pedestal (4)
- # portal (19)
- # quil (3)
- # reagent (8)
- # remote-jobs (1)
- # shadow-cljs (48)
- # spacemacs (5)
@andreolvr0 this line is wrong. I think that you can open a pull request fixing it
Guess someone already tried that, the last commit on that file is from a https://github.com/pedestal/pedestal/pull/668that removed defroutes and added expand-routes , but just the removal of defroutes landed on master... See the https://github.com/pedestal/pedestal/pull/668#issuecomment-700073719 on the PR, so I'm confused... maybe this is a bug?
this commit introduced the bug https://github.com/pedestal/pedestal/commit/50fe5ea89108998ac5a79a02a44432fd111ea6f8 I think that a PR that only change this line can be merged https://github.com/pedestal/pedestal/blob/master/samples/hello-world/src/hello_world/server.clj#L32 The PR #668 proposed to change port and many other unwanted changes.
But thehttp://pedestal.io/guides/hello-world#_routes uses expand-route :man-shrugging: