'open' package for cross platform url launching

michalbe / jsnews-client

:alien: Commandline client for JS News

http://github.com/michalbe/jsnews-client

MIT License

38 stars 5 forks source link

'open' package for cross platform url launching #83

Closed leriel closed 10 years ago

leriel commented 10 years ago

Dziala pod osx i windows, pod linuxem nie mialem mozliwosci sprawdzenia.

leriel commented 10 years ago

https://github.com/nlf/precommit-hook/commit/ed5e67d62abaa28df922e32f5b769367f102e8c1

precommit-hook zmienilo z "jshint": "^2.4.4" na "jshint": "~2.4.4"

co zrobic z tym fantem? zmienic nasze devdependency na ~2.4.X ?

michalbe commented 10 years ago

@leriel : Zgodziliśmy się wcześniej nie lądować rzeczy bez Code Review, revertuje, zrobie Review i wyląduje jeśli nie będzie zastrzeżeń.

leriel commented 10 years ago

Ok - wybacz, źle zrozumiałem tamten post

michalbe commented 10 years ago

Nie ma sprawy, chodzi mi o co że chcę być na bieżąco z tym co dzieje się w aplikacji, wylądujesz kilkanaście commitów i się pogubię :). Jak sam będę pchać coś to będę Ciebie pytać o R.

michalbe commented 10 years ago

Wyslesz ten PR jeszcze raz? Bo chyba nie można revertowanego jeszcze raz zmergować.

leriel commented 10 years ago

Juz wysyłam

leriel commented 10 years ago

Hmm screen shot 2014-07-24 at 8 07 19 pm

Kiedy robię 'switch the base' pokazuje mi takie zmiany jakbym miał wciągnąć Twojego reverta do siebie.. a może jesteś w stanie przywrócić całe repo do e2002529dcee62200f0f6a8b0c24cb0b842608e9 (czyli mój merge) ?

Jeszcze raz przepraszam

leriel commented 10 years ago

Alternatywnie jeśli wolisz, zaciągnę do siebie reverta i wrzucę te zmiany jeszcze raz - ?

michalbe commented 10 years ago

dobra, jakos sobie poradze, chwilka