Fork me on GitHub
#devcards
<
2015-10-14
>
bhauman03:10:51

it doesn't need to be anymore.

bhauman03:10:46

@ricardo: I'm available tomorrow if you need help.

ricardo06:10:23

@bhauman: While looking into dependency issues that might be causing that problem, I noticed that devcards depends on react 0.13.1-0, but sablono depends on a more recent version 0.13.3.-0.

ricardo06:10:10

I was about to submit a pull request syncing them… then realized there’s a few other more recent dependencies.

ricardo06:10:04

Any objections to my sending a PR updating reagent, react, ClojureScript and OM? If not, do I version it as 0.2.0-4?

ricardo10:10:57

Hmpf. So I was about to release this on Gitlab, to give them a fair shake, then they started returning 503. 😞

ricardo14:10:03

@bhauman: Cool, I’ll send it in later today. I take it you’re fine with v 0.2.0-4.

frankhale14:10:02

This isn't high priority or anything but wanted to report it. I tried Clojure 1.8.0-beta1 in my devcards app and running lein figwheel results in the following error: https://gist.github.com/frankhale/add6053284fb955403e1

ricardo15:10:06

@bhauman: PR Submitted. Tested it locally against khroma-tests

bhauman15:10:50

@frankhale: can you make an issue for that?

bhauman15:10:12

@ricardo: So devcards is compiling ok now??

ricardo15:10:26

@bhauman: Yes. It was related to changes on a new namespace, which did not have devcards on :require, just on :require-macros. If cljsbuild had to recompile because just that one had changed, devcards did not end up being included.

bhauman15:10:10

@ricardo: only for that file? or the whole project?

ricardo15:10:47

It did not end up on the final khroma_test .js - I’m compiling with :whitespace.

bhauman15:10:50

@ricardo: that is still not what I would expect

ricardo15:10:10

@bhauman: I can try and repro again later. Did not try to confirm it regressed if I removed it again and then edited just that .cljs - was tired to chasing it.

bhauman15:10:09

@ricardo: yeah no worries it's a good thing to know about. This conditional code thing is going to continue to cause people problems.

bhauman15:10:50

If people have ideas around how to do this better I am open to it.

bhauman15:10:02

I may change the tutorial to be more like (:require [devcards.core :as dc :refer-macros [defcard]]) and then mention conditional inclusion as an option

bhauman15:10:26

If you want the potential of having your cards inline with your prod code then mention the steps to do that. I think that will save a lot of confusion.

ricardo15:10:48

Let me get more familiar with it (and confirm that was indeed the problem) before I comment.

ricardo15:10:25

Since on this project I’m always using it, I haven’t really tried to bring them into a project where I don’t want them on production (although that’ll happen soon).

frankhale15:10:40

is it possible to run 'lein figwheel' and have it start an https server rather than http?

bhauman15:10:54

@frankhale: nope, but you can use a tunneler like https://github.com/jugyo/tunnels

bhauman15:10:14

I need to write a FAQ

frankhale15:10:31

I don't think tunnels will work in my use case

frankhale15:10:41

I need to figure out a way to run a HTTPS server instead of HTTP

frankhale15:10:27

my app requests a secure api, injects an iframe and I need to inspect the iframe contents and cannot do that since my original protocol HTTP is different from the injected iframes protocol HTTPS

frankhale15:10:33

is figwheel using ring under the hood to start the server up?

frankhale15:10:47

I'm digging into the code now

frankhale16:10:01

I found the code

frankhale16:10:18

the princess I'm looking for isn't in that castle

frankhale16:10:28

I guess I need to set up a HTTPS server like nginx or lighttpd in front and proxy to my devcards app so I can make this work

ricardo16:10:59

Oh by the way @bhauman, khroma-tests is where I get that rendering aberration you pointed out on Twitter.

bhauman16:10:15

@frankhale: that is what tunnels is. It's an HTTPS proxy

bhauman16:10:58

@ricardo: very cool, I'll have to check it out and load em up!!

ricardo16:10:20

@bhauman: Confirming the build issue - it’s not a Heisenbug. If I go to khroma-tests.runtime and remove the devcards.core reference from :require, it does not get included on the resulting .js

ricardo16:10:12

@bhauman: Even further… if I remove it from other namespaces (tried storage and browser-action) build does go through correctly. It’s something with runtime.

bhauman16:10:45

@ricardo: cool! thanks, I will keep that in mind when I look at it

bhauman16:10:52

super helpful

ricardo16:10:59

This is all while lein chromebuild auto is running.

frankhale16:10:36

@bhauman: can't actually try tunnels yet because I can't get a working Ruby dev environment to install and work on Windows

frankhale16:10:47

can't build native modules

bhauman16:10:29

sorry for the goose chase

frankhale17:10:31

nah, I'm at a dead stop right now so I have plenty of time to troubleshoot this ruby install 😃