PIStringTests #55

Closed
Gama wants to merge 7 commits from PIStringTests into master
Collaborator
No description provided.
Gama added 6 commits 2021-01-21 11:50:45 +03:00
Gama added 1 commit 2021-01-21 11:53:59 +03:00
Author
Collaborator

Были написаны тесты для PIStringList, PIByteArray. Доделаны тесты для PIString.

Были написаны тесты для PIStringList, PIByteArray. Доделаны тесты для PIString.
zzuummaa requested review from andrey 2021-01-21 12:43:10 +03:00
zzuummaa added the Тестируется label 2021-01-21 12:43:17 +03:00
Contributor

Такой pull-request просмотреть нереально. Предлагаю создать ветку от мастера и переносить туда по 10 тестов и делать pull-request с 10 новыми тестами

Кроме того в изменения попали тесты PIByteArray

Предлагаю для каждого pull-request-а:

  1. Выкачивать актуальные изменения и переключаться на ветку master
  2. Создавать новую ветку с названием, отражающим суть изменений
  3. Переключаться на ветку из п.2
  4. Копировать туда 10 тестов из ветки PIStringTests
  5. Проверять, что работает
  6. Комитить, пушить
  7. Создавать pull-request с мастер
  8. Если есть замечания, исправлять их, делая комиты в ветку из п.2
  9. После принятия pull-request-а удалять ветку из п.2

И делать так, пока не сольем все тесты с мастер

Такой pull-request просмотреть нереально. Предлагаю создать ветку от мастера и переносить туда по 10 тестов и делать pull-request с 10 новыми тестами Кроме того в изменения попали тесты PIByteArray Предлагаю для каждого pull-request-а: 1. Выкачивать актуальные изменения и переключаться на ветку master 2. Создавать новую ветку с названием, отражающим суть изменений 3. Переключаться на ветку из п.2 4. Копировать туда 10 тестов из ветки PIStringTests 5. Проверять, что работает 6. Комитить, пушить 7. Создавать pull-request с мастер 8. Если есть замечания, исправлять их, делая комиты в ветку из п.2 9. После принятия pull-request-а удалять ветку из п.2 И делать так, пока не сольем все тесты с мастер
zzuummaa added Отложено and removed Тестируется labels 2021-01-21 17:02:19 +03:00
Owner

Предлагаю данный ПР закрыть

Предлагаю данный ПР закрыть
andrey declined to review 2021-01-22 10:29:43 +03:00
andrey closed this pull request 2021-01-27 14:22:02 +03:00

Pull request closed

Sign in to join this conversation.