Skip to content
This repository has been archived by the owner on Apr 24, 2018. It is now read-only.

Add tests for data/transform.js and data/dataIO.js #56

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

MaxymVlasov
Copy link
Member

No description provided.

@MaxymVlasov MaxymVlasov self-assigned this Dec 28, 2016
@MaxymVlasov MaxymVlasov requested a review from webknjaz December 28, 2016 23:41
@MaxymVlasov MaxymVlasov force-pushed the tests branch 2 times, most recently from c225470 to 8fea083 Compare December 28, 2016 23:48
@webknjaz
Copy link
Member

Дарма ти випилював статистики, конфлікт буде. Я після мерджа #52 гляну :)
Поки раджу зробити git rebase --interactive master, щоб гілку причесати

@webknjaz
Copy link
Member

webknjaz commented Dec 29, 2016

@MaxymVlasov раджу почитати статті про коміти, що я Філіпу казав:

@MaxymVlasov
Copy link
Member Author

@webknjaz Хоч сказати, що від додавання і ревертання коміту - буде конфлікт?
Ти точно нічого не плутаєш?

Щодо rebase - мені fixup’ом ревертнуті коміти (і їх реверти) попричипляти до попередніх? Чи ти мав на увазі тільки коміт меседжі?

@webknjaz
Copy link
Member

там можна через d їх взагалі видалити, якщо анулюють один одного

@webknjaz
Copy link
Member

так, буде конфлікт при ребейзі. нещодавно таке на власні очі бачив. через те, що коміти один за одним накладаються

@MaxymVlasov
Copy link
Member Author

Та я тоді мерджити збирався. Ок

@MaxymVlasov
Copy link
Member Author

MaxymVlasov commented Dec 29, 2016

@webknjaz git rebase -i не вміє в d

git version 2.1.4

@webknjaz
Copy link
Member

як це не вміє? дуже навіть вміє. то ти просто мабуть забув захопити на 1 коміт більше, чи не знав, що не можна видалити тільки перший коміт, який в списку

@MaxymVlasov
Copy link
Member Author

➜  events-parser git:(tests) git rebase -i HEAD~25
Unknown command: d ...
Please fix this using 'git rebase --edit-todo'.
➜  events-parser git:(2f0e7cf)
# Rebase e8290a0..d5924d6 onto e8290a0
#
# Commands:
#  p, pick = use commit
#  r, reword = use commit, but edit the commit message
#  e, edit = use commit, but stop for amending
#  s, squash = use commit, but meld into previous commit
#  f, fixup = like "squash", but discard this commit's log message
#  x, exec = run command (the rest of the line) using shell

Так що нема такої команди

@MaxymVlasov
Copy link
Member Author

Все набагато простіше: треба видалити рядок коміту
https://gist.github.com/obfk/5258119

@webknjaz
Copy link
Member

аа, ну так значить в тебе старющий гіт, онови його)

@MaxymVlasov
Copy link
Member Author

MaxymVlasov commented Dec 29, 2016

apt-get - 2.1 найновіша
tar.gz - помилка в make-файлі
git clone не має сенсу без make

😱

@webknjaz
Copy link
Member

ох вже ці ваші дебіани)

$ git --version
git version 2.9.0

(і це вже мабуть теж стара)
раджу читати новини про нові фічі в гіті у блозі Атлассіана, там основні показують зазвичай

@webknjaz
Copy link
Member

https://packages.debian.org/search?keywords=git
для дебіана є 2.11

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants