Skip to content

Vcs#8

Open
ASGusev wants to merge 56 commits intomasterfrom
vcs
Open

Vcs#8
ASGusev wants to merge 56 commits intomasterfrom
vcs

Conversation

@ASGusev
Copy link
Owner

@ASGusev ASGusev commented Mar 23, 2017

No description provided.

@jzuken
Copy link

jzuken commented Apr 6, 2017

хорошо бы где-то показывать текущий бранч

@jzuken
Copy link

jzuken commented Apr 6, 2017

и если нет изменённых файлов, то новый коммит делать не стоит

@jzuken
Copy link

jzuken commented Apr 6, 2017

✗ java -jar VCS.jar log
ID Author Date Time Message
0 me 2017-04-06 16:49:56 Initial commit.
1 me 2017-04-06 16:50:18 1
2 me 2017-04-06 16:50:22 2
✗ java -jar VCS.jar branch create asd
✗ java -jar VCS.jar log
ID Author Date Time Message
2 me 2017-04-06 16:50:22 2

куда делось начало лога? повторяемость 100%

@jzuken
Copy link

jzuken commented May 9, 2017

краткое описание внутреннего устройства не нашёл.
-0.5

@jzuken
Copy link

jzuken commented May 9, 2017

сделал ./gradlew jar, проинициализировал репозиторий, сказал в нём clean и получил
Exception in thread "main" java.lang.IllegalArgumentException: No enum constant Main.Command.clean
at java.lang.Enum.valueOf(Enum.java:238)
at Main$Command.valueOf(Main.java:28)
at Main.main(Main.java:13)
-0.5

@jzuken
Copy link

jzuken commented May 9, 2017

✗ java -jar ../VCS.jar branch create asd
✗ java -jar ../VCS.jar checkout branch asd
✗ java -jar ../VCS.jar log
Commits in branch master:
ID Author Date Time Message
1 me 2017-05-09 12:39:49 1
2 me 2017-05-09 12:42:29 2

у меня так и не получилось бранч сменить хоть как-нибудь
-0.5

UPD: а нет, файлы на диске он меняет, но команда лог выдаёт одно и то же всегда

@jzuken
Copy link

jzuken commented May 20, 2017

✗ java -jar ../VCS.jar checkout commit asd
Unknown error:For input string: "asd"

непонятно, чего в ней unknown, я просто попытался сделать чекаут на несуществующий коммит

@jzuken
Copy link

jzuken commented May 20, 2017

модульных тестов всё ещё не вижу

@ASGusev
Copy link
Owner Author

ASGusev commented May 26, 2017

Тест на конфликты слияния оставил интеграционным, остальное сделал модульным.

@jzuken
Copy link

jzuken commented May 29, 2017

ок. 7 баллов за первую задачу, 9.5 за вторую

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants