версия 1#21
Conversation
robotomize
left a comment
robotomize
left a comment
There was a problem hiding this comment.
Все отлично, но есть несколько моментов, которые нужно исправить
Для gitignore тебе нужно создать файлик .gitignore и посмотреть как добавить туда файлы и папки, чтобы лишние файлы не улетали в репозиторий
материалы
| Author: supppppppbro <113025477+supppppppbro@users.noreply.github.com> | ||
| Date: Wed Nov 22 17:35:16 2023 +0300 | ||
|
|
||
| Delete precode directory |
There was a problem hiding this comment.
Вот все такие файлы лишние надо удалить, чтобы их не было в репозитории
| return fmt.Errorf("read file: %w", err) | ||
| } | ||
|
|
||
| if err = yaml.Unmarshal(yamlData, &dockerCompose); err != nil { |
There was a problem hiding this comment.
Тут предполагается автором, что у нас в стурктуре YAMLData и JSONData уже есть стурктур *models.DockerCOmpose нужно анмаршаллить прям в нее &j.DockerCompose
| return fmt.Errorf("read file: %w", err) | ||
| } | ||
|
|
||
| if err = json.Unmarshal(jsonData, &dockerCompose); err != nil { |
There was a problem hiding this comment.
Тут предполагается автором, что у нас в стурктуре YAMLData и JSONData уже есть стурктур *models.DockerCOmpose нужно анмаршаллить прям в нее &j.DockerCompose
| func (j *JSONData) Encoding() error { | ||
| // ниже реализуйте метод | ||
| // ... | ||
| var dockerCompose models.DockerCompose |
There was a problem hiding this comment.
Это код теперь стал не актуальным, надо удалить, мы анмаршалим сейчас прям в структуру j.DockerCompose
robotomize
left a comment
robotomize
left a comment
There was a problem hiding this comment.
Отличная работа! В следущих работах не забудь удалять лишние файлы!
2 participants
No description provided.