Prośby o ściągnięcie (pull)

Platforma Dynamo polega na kreatywności i zaangażowaniu społeczności, więc zespół dodatku Dynamo zachęca współtwórców do badania możliwości, testowania pomysłów i zbierania opinii w społeczności. Mimo że zachęcamy do tworzenia innowacji, zmiany zostaną scalone tylko w przypadku, gdy ułatwiają korzystanie z dodatku Dynamo i spełniają wytyczne zdefiniowane w tym dokumencie. Zmiany zapewniające jedynie niewielkie korzyści nie zostaną scalone.

Oczekiwania dotyczące próśb o ściągnięcie (pull)

Zespół dodatku Dynamo oczekuje, że prośby o ściągnięcie (pull) będą zgodne z następującymi wskazówkami:

• Przestrzegaj naszych standardów kodowania i standardów nazewnictwa węzłów.

• Uwzględnij testy jednostkowe podczas dodawania nowych funkcji.

• Podczas naprawiania błędu dodaj test jednostkowy wskazujący, że bieżące zachowanie jest niewłaściwe.

• Skoncentruj się na jednym problemie. Jeśli pojawi się nowe lub powiązane zagadnienie, utwórz nowy problem.

Oto kilka wskazówek dotyczących tego, czego nie należy robić:

• Zaskakiwanie nas, prośbami o ściągnięcie (pull) zmian o dużych rozmiarach. Zamiast tego należy zgłosić problem i rozpocząć dyskusję, aby można było uzgodnić kierunek prac przed zainwestowaniem dużej ilości czasu.

• Zatwierdzanie (commit) kodu, którego nie napisano samodzielnie. W przypadku znalezienia kodu, który warto byłoby dodać do dodatku Dynamo, przed kontynuowaniem należy zgłosić problem i rozpocząć dyskusję.

• Przesyłanie próśb o ściągnięcie (pull), które zmieniają nagłówki lub pliki związane z licencjami. Jeśli uważasz, że jest z nimi jakiś problem, zgłoś to, a chętnie to omówimy.

• Dodawanie elementów w interfejsie API bez zgłaszania problemu i omówienia tego z nami.

Wypełnianie szablonu prośby o ściągnięcie (pull)

W przypadku przesyłania prośby o ściągnięcie (pull) należy użyć domyślnego szablonu prośby o ściągnięcie. Przed przesłaniem prośby o ściągnięcie (pull) należy się upewnić, że cel został jasno opisany i wszystkie poniższe stwierdzenia można uznać za prawdziwe:

• Po przesłaniu tej prośby o ściągnięcie (pull) baza kodu będzie w lepszym stanie

• Rozwiązanie udokumentowano zgodnie ze standardami

• Poziom przetestowania rozwiązania, którego dotyczy ta prośba o ściągnięcie (pull), jest odpowiedni

• Jeśli istnieją ciągi widoczne dla użytkownika, są one wyodrębnione do plików *.resx

• Wszystkie testy zostały zakończone pomyślnie z wykorzystaniem samoobsługowej ciągłej integracji

• Migawka zmian interfejsu użytkownika, jeśli takie zmiany istnieją

• Zmiany w interfejsie API są zgodne z semantyczną obsługą wersji i są opisane w dokumencie zmian w interfejsie API.

Zespół Dynamo przypisze do prośby o ściągnięcie (pull) odpowiedniego recenzenta.

Proces weryfikacji prośby o ściągnięcie (pull)

Po przesłaniu prośby o ściągnięcie (pull) może być konieczne dalsze zaangażowanie w proces weryfikacji. Należy pamiętać o następujących kryteriach przeglądu:

• Zespół dodatku Dynamo spotyka się raz w miesiącu, aby przejrzeć prośby o ściągnięcie (pull), od najstarszych do najnowszych.

• Jeśli w przypadku przejrzanej prośby o ściągnięcie (pull) wymagane jest wprowadzenie zmian przez jej właściciela, ten właściciel ma 30 dni na odpowiedź. Jeśli dla danej prośby o ściągnięcie (pull) nie będzie żadnej aktywności do następnej sesji, zostanie ona zamknięta przez zespół lub w zależności od jej przydatności zostanie przejęta przez kogoś z zespołu.

• Do przesyłania próśb o ściągnięcie (pull) należy używać domyślnego szablonu prośby o ściągnięcie dodatku Dynamo.

• Prośby o ściągnięcie (pull), dla których nie wypełniono całkowicie szablonów próśb o ściągnięcie dodatku Dynamo z podaniem wszystkich deklaracji, nie zostaną przejrzane.

Wybieranie zatwierdzeń dodatku Dynamo Revit

Ponieważ na rynku dostępnych jest wiele wersji programu Revit, może być konieczne wprowadzenie zmian w określonych gałęziach wersji dodatku DynamoRevit, tak aby różne wersje programu Revit mogły korzystać z nowych funkcji. Podczas procesu przeglądu współtwórcy odpowiadają za wybranie ich zweryfikowanych zatwierdzeń (commit) w innych gałęziach dodatku DynamoRevit zgodnie z wytycznymi zespołu Dynamo.