Conversation
test
Add information about people
Add information about people
Add information about people
There was a problem hiding this comment.
- Заметила, что ссылки на фотографии не везде корректны (например: фото 39, 3). Необходимо пересмотреть все ссылки и исправить их на корректные
- Насколько я знаю, Марк Дмитриевич отправил Вам перечень сайтов, с которых нужно взять фотографии. Почему-то часть фотографий взята с сайтов, которые отсутствуют в перечне. Удалите такие фотографии и замените их на изображения с указанных сайтов. Если найденное изображение не соответствует требованиям из документа, то просто проигнорируйте его, не пытайтесь найти фото этого музыканта в другом источнике
- Все студенты собирали 100 фотографий, у Вас всего 92. Добейте, пожалуйста, до нужного количества
|
https://github.com/Tony-StPet/orchestra_musician |
|
Что Вы подразумеваете под "папкой в древе?" Речь про ветку? |
|
@Tony-StPet
|
|
Переделал, согласно заданию, ветвь orchestra_musician |
|
Посмотрела изображения из readme (36 картинок). То, что нужно! Теперь буду ждать оформления файла с описанием |
|
50 изображений и файл описание. Готово. Если будет время, могу увеличить количество до 100. все зависит от МД |
|
На будущее: очень рекомендую Вам делать информативные сообщения к коммитам ("add" не подойдет, сами потом запутаетесь). Это хорошая привычка, не игнорируйте этот момент. Это совет, не замечание. Насчет работы: все ок, проверила описание. Насчет кол-ва фото последнее слово будет за Марком Дмитриевичем, я проверяю только качество выполнения задания @markpolyak, ситуация следующая:
Итого: не соблюдены требования по работе в GitHub, к фотографиям претензий нет. |
|
@KleinerKaktus455 работа сдана, пул-реквест можно мерджить |
|
@markpolyak , это не его пул, мерджить нечего, т.к. отдельная ветка не создана( Потом самостоятельно залью изменения |
No description provided.