Fork me on GitHub
#proton
<
2016-01-26
>
dvcrn01:01:30

@geksilla: it’s not linter. Even with disabled linter I am getting the lagg

dvcrn08:01:05

hmm what do you think should we focus on next?

dvcrn08:01:17

should we tackle the automated readmes or are there other more important features?

geksilla09:01:01

readme would be great

geksilla09:01:21

what do you think about option to choose input provider?

geksilla09:01:12

select between vim, emacs or none

austincrft15:01:53

Did you see the updates to Proto REPL? That package seems to be coming along really nicely

geksilla15:01:34

proto-repl integrated with proton check out Readme for more info about key bindings https://github.com/dvcrn/proton/blob/master/src/cljs/proton/layers/lang/clojure/README.md#proto-repl

aurelian15:01:14

would be nice if Proto REPL could connect to existing repl instance, like vim-fireplace does

aurelian15:01:33

just saying

geksilla15:01:43

proto-repl + ink looks very exciting!

sglyon15:01:12

That’d be cool simple_smile (I’m quite familar with ink from my Julia work)

sglyon15:01:30

Is someone working on that ?

geksilla16:01:17

you mean using ink in proto-repl?

geksilla16:01:19

yes, initial work is done

sglyon16:01:49

Cool. Do you know if they want to use the ink console to house the REPL, or just use ink for inline stuff?

geksilla16:01:51

you can activate it by switching config proto-repl.showInlineResults to true

geksilla16:01:47

just saw few words about ink in changelog

sglyon16:01:53

Ahh ok cool.

sglyon16:01:54

no problem, I was just curious

meow16:01:05

I did a fresh install of Atom on Windows 10, then installed proton-mode and it appears to be hung. 😞

geksilla16:01:52

looks like cannot create .proton file

geksilla16:01:33

I thought that I fixed that(

geksilla16:01:08

can you check USERPROFILE variable in your env?

meow16:01:32

I just shut it down.

meow16:01:23

How do I check that on Windows 10? I'm on a borrowed laptop.

geksilla17:01:30

or you can run atom in safe mode, open dev tools and check for process.env value from develepor tools console

geksilla17:01:55

also since atom uses node 4.x we can use os.homedir()

meow17:01:12

Windows PowerShell
Copyright (C) 2015 Microsoft Corporation. All rights reserved.

PS C:\Users\patri> $env:USERPROFILE
C:\Users\patri
PS C:\Users\patri>

meow17:01:55

PS C:\Users\patri> cd .atom
PS C:\Users\patri\.atom> dir


    Directory: C:\Users\patri\.atom


Mode                LastWriteTime         Length Name
----                -------------         ------ ----
d-----        1/26/2016  10:41 AM                .apm
d-----        1/26/2016  10:41 AM                .node-gyp
d-----        1/26/2016  10:56 AM                blob-store
d-----        1/26/2016  10:41 AM                compile-cache
d-----        1/26/2016  10:41 AM                packages
d-----        1/26/2016  10:39 AM                storage
-a----        1/26/2016  10:39 AM             46 .gitignore
-a----        1/26/2016  10:39 AM            117 config.cson
-a----        1/26/2016  10:39 AM            397 init.coffee
-a----        1/26/2016  10:39 AM           1031 keymap.cson
-a----        1/26/2016  10:39 AM            719 snippets.cson
-a----        1/26/2016  10:39 AM            794 styles.less

geksilla17:01:45

ok, can you launch atom run dev tools and execute following snippet in console:

require('path').normalize(process.env.USERPROFILE);

geksilla17:01:17

you can open dev tools by hitting Ctrl-shift-p and type dev tools

geksilla17:01:27

even when proton hangs

meow17:01:58

I'm brand new to atom. Don't know how to launch atom run dev tools

geksilla17:01:12

launch atom

geksilla17:01:26

hit Ctrl-shift-p to start Command Palete

geksilla17:01:50

and type dev tools

geksilla17:01:27

you will see command named Window: Toggle Dev Tools select it by enter

geksilla17:01:52

now in dev tools select console tab

geksilla17:01:27

click on &gt;&gt;

geksilla17:01:44

click on arrows

geksilla17:01:18

you'll see prompt

meow17:01:49

require('path').normalize(process.env.USERPROFILE);
"C:\Users\patri"

geksilla17:01:01

also can you try to execute

require('os').homedir()

meow17:01:32

require('os').homedir()
"C:\Users\patri"

geksilla17:01:16

the last one please

require('path').join(require('os').homedir(), '.proton')

meow17:01:46

require('path').join(require('os').homedir(), '.proton')
"C:\Users\patri\.proton"

geksilla17:01:40

do you use proton from source or installed by package manager?

meow17:01:13

installed by package manager from within atom

meow17:01:38

see first screenshot

geksilla17:01:29

thanks a lot for debugging!

geksilla17:01:50

if you want I can share probably fixed compiled js

geksilla17:01:33

so you can try to change it in ~/.atom/packages/proton-mode/lib/proton.js

geksilla17:01:42

and it should work

meow17:01:57

would rather wait for official release

meow17:01:19

unless you need me to do it for debugging purposes

geksilla17:01:37

yes, it would be cool to check that this fix is working

meow17:01:49

then tell me what to do

geksilla17:01:17

go to C:\Users\patri\.atom\proton-mode\lib\

geksilla17:01:45

and replace original proton.js with downloaded

geksilla17:01:00

and restart atom

meow17:01:54

same error

meow17:01:53

PS C:\Users\patri\.atom\packages\proton-mode\lib> dir


    Directory: C:\Users\patri\.atom\packages\proton-mode\lib


Mode                LastWriteTime         Length Name
----                -------------         ------ ----
-a----        1/26/2016  11:41 AM        1258925 proton.js

geksilla17:01:41

ok, I'll get windows vm and try to fix this issue.

geksilla17:01:09

thanks for help!

geksilla17:01:03

.proton detection should works fine in that proton.js, probably now issue with copying proton template file

meow17:01:28

np - ping me when you think you've got it fixed and I'll try again - no hurry on my part - just want to try it

geksilla17:01:12

you can also try to create .proton file C:\Users\patri\.proton with content from https://github.com/dvcrn/proton/blob/master/plugin/templates/proton.edn

geksilla17:01:16

and restart atom

geksilla18:01:22

just checked on windows 7 patched version works well when you create .proton in your home dir

geksilla18:01:44

need to fix copy call

geksilla18:01:50

another issue during package installation, looks like on windows apm is not available on path, need to check out how it works with BufferedProcess

geksilla19:01:30

install packages works fine, just was very slow on vm

geksilla19:01:37

@meow to run proton all what you need is just to create .proton and fill it with default template

geksilla19:01:34

@meow new patched version https://www.dropbox.com/s/wh6lqto4auwxifs/proton.js?dl=0 manual create of .proton not required

meow19:01:43

I manually created it and it is now installing 23 packages

geksilla19:01:39

I'll send PR with fix

meow20:01:12

how do I get to a repl

dvcrn23:01:54

@geksilla we are using atoms API though to retrieve apm which I think is the absolute path

dvcrn23:01:08

so it should work in windows

dvcrn23:01:06

@meow: check out the proto repl readme and the keybindings setup here simple_smile https://github.com/dvcrn/proton/tree/master/src/cljs/proton/layers/lang/clojure

dvcrn23:01:01

in general - when you are in a clojure file, everything under the SPC m s or , s menu is repl related