Czy odniesienie do błędu / problemu w komunikacie zatwierdzenia jest uważane za dobrą praktykę?


11

Pracuję nad projektem, w którym mamy kontrolę źródła skonfigurowaną do automatycznego zapisywania notatek w narzędziu do śledzenia błędów. Po prostu zapisujemy identyfikator problemu w komunikacie zatwierdzenia, a komunikat zatwierdzenia jest dodawany jako notatka do narzędzia do śledzenia błędów.

Widzę tylko kilka wad tej praktyki. Jeśli kiedyś w przyszłości kod źródłowy zostanie oddzielony od oprogramowania do śledzenia błędów (lub zgłoszone błędy / problemy zostaną w jakiś sposób utracone). Lub gdy ktoś patrzy na historię zmian, ale nie ma dostępu do naszego narzędzia do śledzenia błędów.

Moje pytanie brzmi, czy umieszczenie odniesienia do błędu / problemu w komunikacie zatwierdzenia jest uważane za dobrą praktykę? Czy są jakieś inne wady?

Odpowiedzi:


10

Przyjęliśmy tę praktykę i działa ona dla nas bardzo dobrze. Ścisła integracja między systemem kontroli wersji (VCS) a innymi używanymi przez nas systemami, np. Ciągłą integracją, śledzeniem błędów itp. Jest niezwykle cenna. Jeśli kiedykolwiek coś zmienimy w przyszłości, z pewnością będziemy musieli ocenić skutki uboczne, w tym powiązania między VCS a systemem śledzenia błędów.

Ogólnie uważałbym to za dobrą praktykę. W przypadku niektórych systemów śledzenia dostępne są dodatkowe opcje i narzędzia, na przykład właściwości bugtraq dla Subversion (SVN). Sugeruje to, że wiele osób widzi wartość w tej praktyce.


13

Jeśli chcesz się naprawdę upewnić, że nie stracisz żadnych informacji, nawet jeśli w przyszłości będziesz mógł użyć innego narzędzia do śledzenia błędów lub dane z narzędzia do śledzenia błędów w jakiś sposób znikną, dlaczego nie po prostu wstawić zarówno identyfikatora problemu, jak i krótkiego wyjaśnienia błędu komunikat zatwierdzenia?

Napraw błąd nr 123: aplikacja uległa awarii po zalogowaniu

W dalszym ciągu masz link z historii zatwierdzeń do narzędzia do śledzenia błędów - a jeśli narzędzie do śledzenia błędów będzie kiedykolwiek niedostępne, nadal możesz zobaczyć w historii, o czym był ten konkretny błąd.


W rzeczywistości to robimy, więc nie musimy przełączać się do śledzenia błędów za każdym razem, gdy przeglądamy historię zatwierdzeń.
— Christian P

Okej, po prostu zostawiłbym to tak, jak jest. IMO, to najlepszy sposób, jak to zrobić!
— Christian Specht

1
Tak, dobra uwaga. Ale i tak to założyłem. Sam identyfikator błędu / problemu nie jest wystarczająco dobry z mojego doświadczenia. Patrząc na dziennik zatwierdzeń, wciąż chcesz zobaczyć, o co chodziło w każdym zatwierdzeniu, np. Jaki był główny powód tej zmiany kodu. Czasami komunikat zatwierdzenia jest bardziej techniczny, podczas gdy tekst systemu śledzenia błędów jest bardziej ukierunkowany na użytkowników oprogramowania.
— Manfred

Jest to ogólnie standardowa praktyka, w której również pracowałem, myślę, że to właściwy sposób, aby to zrobić.
— Carson63000,

+1 zawsze to rób! Właśnie zająłem się utrzymaniem projektu, który jest pełen klejnotów, takich jak „mogła to być przyczyna błędu 5423”. Nie mamy dostępu do ich narzędzia do śledzenia błędów.
— Kryptic,

2

Jest to bardzo powszechna praktyka i uważam ją za niezwykle wygodną. Korzystam z TRAC, więc mogę odczytać historię kodu i przejść do zadania, które spowodowało zmianę, lub odczytać historię zadania i przejść do zmian kodu.

„Jeśli kiedyś w przyszłości ...” Jeśli oddzielisz kod od narzędzia do śledzenia błędów, stara historia zmian prawdopodobnie nie będzie już interesująca.


2

Ja również używam tej praktyki i uważam ją za bardzo dobrą. Ale oprócz identyfikatora problemu dodaję krótki opis błędu / funkcji (zwykle tytuł z systemu śledzenia błędów). To często pomaga zaoszczędzić czas, ponieważ nie muszę szukać w systemie śledzenia błędów (ponieważ rozpoznaję zmianę) ORAZ, jak powiedziałeś, jeśli w jakiś sposób stracę system śledzenia błędów, nie jestem całkowicie zgubiony.

Korzystając z naszej strony potwierdzasz, że przeczytałeś(-aś) i rozumiesz nasze zasady używania plików cookie i zasady ochrony prywatności.
Licensed under cc by-sa 3.0 with attribution required.