dirac

Dirac v1.7.2 is out: https://github.com/binaryage/dirac/releases/tag/v1.7.2
danielcompton 2016-05-08T20:23:45.000205Z

Is it expected to see Use of undeclared Var warnings in the Dirac REPL like this?

2016-05-08T20:24:16.000207Z

yes, it is normal nREPL session in this regard

2016-05-08T20:24:56.000208Z

you have to require or load-file to get rid of those warnings

2016-05-08T20:24:56.000209Z

https://github.com/binaryage/dirac-sample

2016-05-08T20:25:35.000211Z

load-file is a special nREPL middleware command, some IDEs support it, for example Cursive has it as command in menu

danielcompton 2016-05-08T20:26:56.000212Z

Cool, gotcha. Thanks!

danielcompton 2016-05-08T20:27:17.000213Z

Also, I noticed that it seemed like source maps weren’t updating when Figwheel recompiled, is that expected/known?

2016-05-08T20:27:58.000214Z

I didn’t notice that, but I don’t have complex-enough project with figwheel + dirac

2016-05-08T20:28:16.000215Z

still have to start using Dirac on some real project myself 😉

2016-05-08T20:29:15.000216Z

by source maps do you mean “source panel”, when you click some file location link and it shows you outdated sources or location?

danielcompton 2016-05-08T20:29:18.000217Z

I noticed http://dev.clojure.org/jira/browse/CLJS-894, but need to check more to see how normal devtools behaves

danielcompton 2016-05-08T20:29:42.000218Z

Yeah, I noticed it with a (js-debugger) breakpoint set, but I think it will apply anywhere

2016-05-08T20:32:20.000221Z

I was using it to help with outdated source-maps, it was an issue when I was working on Atom package, seems not needed for normal webdev workflows tough

danielcompton 2016-05-08T20:33:24.000222Z

yeah, just trying it now, I don’t think it’s a dirac issue, just devtools

2016-05-08T20:34:32.000225Z

I can’t recall what was the issue with those source maps. Please, let us know if you find the problem.

danielcompton 2016-05-08T20:37:07.000226Z

Yep, that fixes it

danielcompton 2016-05-08T20:37:17.000227Z

Probably all figwheel users should have that option enabled

danielcompton 2016-05-08T20:37:20.000228Z

I think?

2016-05-08T20:38:02.000230Z

not sure, this does not seem to be reported often in figwheel or general clojurescript channels

2016-05-08T20:38:14.000231Z

I would guess some caching setting comes into play

2016-05-08T20:38:34.000232Z

it should be fixed if you force DevTools to not cache anything at all (when open)

2016-05-08T20:38:53.000233Z

that would be an alternative solution I believe

2016-05-08T20:39:25.000234Z

do you use figwheel’s web server for dev or your own?

2016-05-08T20:39:39.000235Z

what headers does it send?

danielcompton 2016-05-08T20:44:59.000236Z

Using figwheels

danielcompton 2016-05-08T20:45:37.000237Z

@darwin: I suspect that may be because not many people use source-maps? :simple_smile:

danielcompton 2016-05-08T20:48:10.000238Z

Nope, even with caching off it still fails

2016-05-08T20:48:30.000240Z

hmm

danielcompton 2016-05-08T20:49:01.000241Z

This is just a general issue, but I suspect most people haven’t hit it much because without dirac, source maps are not quite so useful

danielcompton 2016-05-08T20:49:26.000242Z

They obviously have use, but dirac lets you use them in an interactive fashion much easier

2016-05-08T20:49:39.000243Z

indeed

2016-05-08T20:51:20.000244Z

there is a chance I could be able to fix this in DevTools if this is really a devtools bug

2016-05-08T20:51:49.000245Z

but first let’s wait for more reports for other users as well

2016-05-08T20:53:20.000246Z

Dirac is pretty much unknown to most people and those who are aware of it, they haven’t integrated it yet into their workflows. It will take few months I guess before we start seeing reports like this.

danielcompton 2016-05-08T20:54:08.000248Z

Sure thing

2016-05-08T21:04:02.000249Z

@danielcompton: thanks for the PRs for docs improvements, much appreciated

danielcompton 2016-05-08T21:04:37.000250Z

np, is it ok to leave commit and PR messages blank? I was in a bit of a hurry, but I can fill them out more in the future if it’s helpful

danielcompton 2016-05-08T21:04:47.000251Z

In fact I will fill them out more

danielcompton 2016-05-08T21:04:54.000252Z

Good practice

2016-05-08T21:05:47.000253Z

It is fine. It is pretty obvious that those are grammar/spelling fixes.