DjangoBB LoFi version

Найти - Пользователи

Полная версия: Есть рабочий код, нужна критика и дельные советы опытных.

Начало » Центр помощи » Есть рабочий код, нужна критика и дельные советы опытных.

1 2 3 4 5

Shaman

Сен. 18, 2015 08:07:33

Извращенцы… Наверное стоило или опубликовать код, или обсуждать его в привате.

J.R.

Сен. 18, 2015 08:16:24

J.R.

Сен. 18, 2015 09:12:12

py.user.next

Сен. 18, 2015 10:31:11

Shaman
Наверное стоило или опубликовать код, или обсуждать его в привате.

Не, надо привыкать работать по-нормальному.

J.R.
Если не против, я ссылку на гист опубликую. А то правда нехорошо выходит.

Да можешь публиковать, главное, чтобы каши-малаши не было. Участвовать-то могут многие.

J.R.

TypeError: if you give only one argument to maketrans it must be a dict

Я отменил там. Это я перепутал с list. Ты тогда там сделал list для передачи в dict, тогда как его делать не надо, потому что dict() берёт любой итерабл.

Вот теперь, прежде чем выгружать свои изменения, загружаешь мои через fetch и смотришь, изменилось ли что-нибудь. Если изменилось, свои припрятываешь (stash), мои вливаешь и свои вытаскиваешь (stash apply).

Там, кстати, ветки ещё есть, через github их не видно, но их можно делать и выгружать. Это если идея какая-нибудь возникла, в ветке её делаешь и выгружаешь для оценки, а там можно и влить её, если нормальная, либо просто хорошие комиты выдернуть через черипик.

Второе, что надо рассказать: у тебя слишком много изменений в одном коммите.
Надо делать изменения мелкими, чтобы их можно было отменять либо перебрасывать из ветки в ветку.
Так что если делаешь много изменений, то разделяй их на коммиты путём

git add -p

по такому принципу, когда каждое изменение - это самостоятельный элемент. (Можно добавить, можно убрать, но программа остаётся рабочей.)

J.R.
Вот смотри, склонировал этот гист к себе. Теперь буду с этим колоном работать.

По поводу организации:
У тебя один серверный репозиторий, один локальный репозиторий.
У меня один серверный репозиторий, один локальный репозиторий.
Когда ты хочешь сделать изменения, втягиваешь в свой локальный коммиты из моего серверного, а потом выталкиваешь мои коммиты и свои изменения в свой серверный.
Когда я хочу сделать изменения, втягиваю в свой локальный коммиты из твоего серверного, а потом выталкиваю твои коммиты и свои изменения в свой серверный.

Shaman

Сен. 18, 2015 10:36:46

https://github.com/mrprint
Чуть-чуть возни с гитхабом и visualstudio. Можно критиковать!

J.R.

Сен. 18, 2015 11:10:04

py.user.next

Сен. 18, 2015 12:35:08

J.R.
Вот смотри, я сделал git fetch… а где и как смотреть, что изменилось у тебя?

Если она загрузила что-нибудь, то это находится в удалённой (remote) ветке.

git branch -a

Обращение к удалённой ветке делается через источник/ветка.
К примеру, чтобы влить удалённую ветку в текущую:

git merge fork/master

где fork - имя источника, а master - ветка мастер в том источнике.

А так можно посмотреть, что входит в правую ветку, но не входит в левую:

git log fork/master..origin/master

У меня сейчас моя мастер ветка содержит три коммита, которых нет в твоей мастер ветке, поэтому для просмотра этих коммитов я твою ставлю слева, а свою - справа.

[guest@localhost cuetool-concept-my]$ git log fork/master..origin/master
commit 714baac47bcc8d2c0a330c7cf010c04d166a2828
Author: guest <guest@localhost>
Date:   Fri Sep 18 17:58:30 2015 +1000

    Make static methods dynamic
    
    Use static method when you are going to use it
    without object.
    
    >>> class A:
    ...     @staticmethod
    ...     def f(arg):
    ...         print(arg)
    ...
    >>> a = A()
    >>> a.f(1)
    1
    >>> A.f(2)
    2
    >>>
    
    In other cases use a simple dynamic method.

commit abaee69506c00ed511ba402430373af4305a676c
Merge: 4f5064b 0f58dc2
Author: guest <guest@localhost>
Date:   Fri Sep 18 17:16:59 2015 +1000

    Merge remote-tracking branch 'fork/master'

commit 4f5064b6ce0c601d9517889fcb86764522970eb6
Author: guest <guest@localhost>
Date:   Fri Sep 18 17:16:01 2015 +1000

    Cancel c3ea852
(END)

J.R.
Как мне теперь отменить?

Ты просто вливаешь, я отменил - это значит, что я исправил обратно и закоммитил это исправление.

J.R.
С гитом вопросов больше чем с Питоном…

Это с непривычки, но изучить его надо, есть одна книга бесплатная, её прочитаешь - всё будешь знать, а дальше там только хелп останется читать.

J.R.
По коду замечания есть?

Ну, я там закоммитил уже одну вещь. :)
Сейчас разберёшься, вольёшь и посмотришь.

4kpt_III

Сен. 18, 2015 12:47:01

1. awersome_slugify.
2. docstring.

Без 2 даже читать не стал

J.R.

Сен. 18, 2015 13:15:00

J.R.

Сен. 18, 2015 13:18:46