Fork me on GitHub
#cljsjs
<
2016-04-06
>
martinklepsch16:04:33

@leontalbot: cant find the channel we just wrote in. feel free to ask more q here

leontalbot17:04:23

The author of tabletop.js does not provide .min.js files

leontalbot17:04:31

do I minify myself?

martinklepsch17:04:42

@leontalbot: there's a minify task

martinklepsch17:04:39

gotta run right now but it's in the same namespace as the download task. you can read the help if you run boot minify -h (assuming you have something like :refer [minify])

leontalbot18:04:24

was I suppose to commit with the target dir in the tabletop folder? cause I removed it to do like the rest of the libs

leontalbot18:04:11

"No test commands were found"

leontalbot18:04:19

I would need help to understand what to do...

juhoteperi18:04:47

@leontalbot: Your branch is based on very old packages commit (680 commits behind master), so your branch doesn't contain the CI configuration

leontalbot18:04:44

I went to https://github.com/cljsjs/packages and clicked fork... What should I have done?

juhoteperi18:04:01

It is probably some months since you forked it

juhoteperi18:04:47

Hmm, shouldn't happen. Anyway, you probably have currently one git remote set up, probably named origin and it points to your fork in github.

juhoteperi18:04:56

you can check this using git remote show origin

leontalbot18:04:35

* remote origin
  Fetch URL: 
  Push  URL: 
  HEAD branch: master
  Remote branches:
    foreign-libs        tracked
    foreign-libs-link   tracked
    highlight           tracked
    issue-47            tracked
    master              tracked
    native-closure      tracked
    openlayers-all-exts tracked
    patch-1             tracked
    pdf.js              tracked
    react-0.13          tracked
    tabletop            tracked
    test-utils-extern   tracked
  Local branches configured for 'git pull':
    master   merges with remote master
    tabletop merges with remote tabletop
  Local refs configured for 'git push':
    master   pushes to master   (up to date)
    tabletop pushes to tabletop (up to date)

juhoteperi18:04:40

You can add second git remote using git remote add upstream [email protected]:cljsjs/packages.git and then you can use this upstream remote to load latest changes into your local copy

juhoteperi18:04:47

something like git checkout master (move to local master branch), git pull upstream master (update local master branch to match upstream remote), git checkout tabletop (move back to feature branch) and git merge master (merge changes from master back to the feature branch)

leontalbot18:04:07

great ! trying...

leontalbot18:04:39

git pull upstream master
Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

juhoteperi18:04:59

Oh right, you should use https url for upstream repo as you don't have commit rights

juhoteperi18:04:30

git remote set-url upstream

leontalbot18:04:01

Pull request just sent!

juhoteperi18:04:53

@leontalbot: Commented on PR about the extern file

leontalbot18:04:20

@juhoteperi to make sure I do the extern correctly...

leontalbot18:04:28

var Tabletop = {
    "callbacks": function () {},
    "init": function () {},
    "sheets": function () {},
    "Model": function () {}
}

juhoteperi18:04:57

@leontalbot: Yes, something like that

leontalbot19:04:01

pull-requested!

leontalbot19:04:42

@juhoteperi hope the externs look good now simple_smile

leontalbot22:04:20

@juhoteperi: thanks for the merge!