Hacker News Board Hacker News, browsed like a classic message board
Live data from Hacker News

Eric Wong on why Unicorn will not be hosted on GitHub

bogomips.org

Post reply on HN
Print view View original

41–48 of 48 posts

Re: Eric Wong on why Unicorn will not be hosted on GitHub

#41

Post by __david__ »

yeukhon wrote: post #40
__david__ wrote: post #26

Earlier quoted context omitted.

> Github can shut down next year all the sudden and you can lose everything… This is entirely false, if github shuts down tomorrow you don't lose everything. In fact, most likely you don't lose anything , since the git repo on your computer has everything. This is the only reason I use Github, myself. It's not like the old days of Sourceforge hosted CVS servers where you didn't actually have access to your data.

As someone else already mentioned, this is not entirely false. You will lose your issues which is a huge reason why big projects don't move to Github. You lose your wikis. You don't run backup of your issues every day, do you? In fact, how many of you do that? I bet the number is almost zero. Source code is one thing, but the history of discussion? That worth something. Also, sometimes some repos just don't exist on…

I don't care about issues or wikis. I don't use either feature in the projects that originate from me (people send me issues but I don't use them to organize development). The code is the important part. Any discussion is already backed up in my email. And even if I did care about issues I could download them through the api, as was pointed out elsewhere.

Re: Eric Wong on why Unicorn will not be hosted on GitHub

#42

Post by __david__ »

lomnakkus wrote: post #32
erikb wrote: post #27

Earlier quoted context omitted.

Git is one of these tools you don't use right if you don't use it in the shell. Otherwise it's like trying to fly a submarine. You might make it work if you attach enough stuff, but still it's better to use it under water or to use a real aircraft. And seriously, if you use it in the shell there is not the slightest need for Github. I love Github, but there is no additional information you gain from using it. It's ju…

False dichotomy. It's quite possible to use both. One thing that git stinks at is line-by-line code review, for example. While I don't necessarily think GitHub is especially good for that, it does do the job in a much simpler way. (Yes, I'm aware of the practice of mailing patches for review, but you're not going to convince very many people that it's better than a GUI-type thing. It also doesn't address stuff like a…

> One thing that git stinks at is line-by-line code review, for example.

Really? That's one thing I haven't ever seen anyone do better than plain old 'git log -p'. I use this alias:

  [alias]
      review = log -p --reverse
After `git fetch`-ing I copy the hash range that I fetched and paste it after `git review`. Then in Less I can type "/^diff" to use n and p to skip back and forth between commits. I find this is much easier to read than Github's html diff views and infinitely more efficient than the millions of clicks I need in web browser to do anything (plus there's absolutely no lag, unlike the browser).

Re: Eric Wong on why Unicorn will not be hosted on GitHub

#43

Post by lomnakkus »

__david__ wrote: post #42
lomnakkus wrote: post #32

Earlier quoted context omitted.

False dichotomy. It's quite possible to use both. One thing that git stinks at is line-by-line code review, for example. While I don't necessarily think GitHub is especially good for that, it does do the job in a much simpler way. (Yes, I'm aware of the practice of mailing patches for review, but you're not going to convince very many people that it's better than a GUI-type thing. It also doesn't address stuff like a…

> One thing that git stinks at is line-by-line code review, for example. Really? That's one thing I haven't ever seen anyone do better than plain old 'git log -p'. I use this alias: [alias] review = log -p --reverse After `git fetch`-ing I copy the hash range that I fetched and paste it after `git review`. Then in Less I can type "/^diff" to use n and p to skip back and forth between commits. I find this is much easi…

How are you gathering up comments and making sure that all issues are addressed, etc.?

Re: Eric Wong on why Unicorn will not be hosted on GitHub

#44

Post by __david__ »

lomnakkus wrote: post #43
__david__ wrote: post #42

Earlier quoted context omitted.

> One thing that git stinks at is line-by-line code review, for example. Really? That's one thing I haven't ever seen anyone do better than plain old 'git log -p'. I use this alias: [alias] review = log -p --reverse After `git fetch`-ing I copy the hash range that I fetched and paste it after `git review`. Then in Less I can type "/^diff" to use n and p to skip back and forth between commits. I find this is much easi…

How are you gathering up comments and making sure that all issues are addressed, etc.?

By talking to the person, in person or on the phone or IM or email, depending on the priority of the question/issue. If something needs to be addressed then it goes into the bug tracker (whatever that might be). For egregious things, we might even `git revert` the patch and let the person rethink/rework it.

Re: Eric Wong on why Unicorn will not be hosted on GitHub

#45

Post by lomnakkus »

__david__ wrote: post #44
lomnakkus wrote: post #43

Earlier quoted context omitted.

How are you gathering up comments and making sure that all issues are addressed, etc.?

By talking to the person, in person or on the phone or IM or email, depending on the priority of the question/issue. If something needs to be addressed then it goes into the bug tracker (whatever that might be). For egregious things, we might even `git revert` the patch and let the person rethink/rework it.

That doesn't address the actual recording of the comments such that they can easily see the context of your comments. It also doesn't adress pre-merge review and pre-merge integration automated testing.

In short: No. Although I'm happy that your process works for you it doesn't (and won't) work for a lot of other situations (including mine at my current place of work nor my previous one).

Re: Eric Wong on why Unicorn will not be hosted on GitHub

#46

Post by __david__ »

lomnakkus wrote: post #45
__david__ wrote: post #44

Earlier quoted context omitted.

By talking to the person, in person or on the phone or IM or email, depending on the priority of the question/issue. If something needs to be addressed then it goes into the bug tracker (whatever that might be). For egregious things, we might even `git revert` the patch and let the person rethink/rework it.

That doesn't address the actual recording of the comments such that they can easily see the context of your comments. It also doesn't adress pre-merge review and pre-merge integration automated testing. In short: No. Although I'm happy that your process works for you it doesn't (and won't) work for a lot of other situations (including mine at my current place of work nor my previous one).

Recording of comments is pretty pointless. The code is what matters and that is safely in git. Anything really important should go in the commit message, where it belongs.

If the comments are that important to you then you are foolish to use Github's interface because that means that you can only ever use Github to look at the patch and see the comments. And there's never an indication that the patch even has any comments on it in the git repo itself (rendering git log useless).

It would be much better to use Bugzilla (or something similar) the way Mozilla does, with bug numbers referenced in the commits and commits referenced in the bugs.

There's also "git notes" but it has some funky corner cases IIRC.

I also am not understanding what Github's clunky commit viewer has to do with "pre-merge integration automated testing". That sounds like CI territory.

Re: Eric Wong on why Unicorn will not be hosted on GitHub

#47

Post by erikb »

lomnakkus wrote: post #45
__david__ wrote: post #44

Earlier quoted context omitted.

By talking to the person, in person or on the phone or IM or email, depending on the priority of the question/issue. If something needs to be addressed then it goes into the bug tracker (whatever that might be). For egregious things, we might even `git revert` the patch and let the person rethink/rework it.

That doesn't address the actual recording of the comments such that they can easily see the context of your comments. It also doesn't adress pre-merge review and pre-merge integration automated testing. In short: No. Although I'm happy that your process works for you it doesn't (and won't) work for a lot of other situations (including mine at my current place of work nor my previous one).

I don't get your problem with automated testing. In my eyes this is the one feature that the shell has, but Github doesn't. That's why I'm quite confused. You can addd git hooks to automate it, or simply call the corresponding command while you are in the shell already. But GitHub neither builds nor tests your code, it simply shows you the git log, git diff, etc. as HTML.

Re: Eric Wong on why Unicorn will not be hosted on GitHub

#48

Post by lomnakkus »

erikb wrote: post #47
lomnakkus wrote: post #45

Earlier quoted context omitted.

That doesn't address the actual recording of the comments such that they can easily see the context of your comments. It also doesn't adress pre-merge review and pre-merge integration automated testing. In short: No. Although I'm happy that your process works for you it doesn't (and won't) work for a lot of other situations (including mine at my current place of work nor my previous one).

I don't get your problem with automated testing. In my eyes this is the one feature that the shell has, but Github doesn't. That's why I'm quite confused. You can addd git hooks to automate it, or simply call the corresponding command while you are in the shell already. But GitHub neither builds nor tests your code, it simply shows you the git log, git diff, etc. as HTML.

Oh, sorry, I guess I wasn't very clear. I was talking about automated integration testing, i.e. testing on a wholly different system than the developer's machine. (This is to eliminate the "well, it works for me!" problem.)
Post reply on HN