Skip to content

Ошибка чтения настроек, восстановление коммитера#60

Merged
nixel2007 merged 4 commits intonixel2007:developfrom
240596448:develop
Oct 10, 2023
Merged

Ошибка чтения настроек, восстановление коммитера#60
nixel2007 merged 4 commits intonixel2007:developfrom
240596448:develop

Conversation

@240596448
Copy link
Contributor

@240596448 240596448 commented Oct 5, 2023

В продолжение 59

  1. Ошибка получения настроек
    было: git config user.name --local - устанавливало значение настройки --local (т.е. все что получало настройку - работало не верно)
    стало: git config --local user.name

  2. Появилась проблема получения "отсуствующей настройки" - команда возвращает код 1.
    Если просто устанавливать (на время получения настройки) тихий режим, то всех будет пугать в логе сообщение об ошибке.
    Пришлось закостылить исключение кода = 1. Если есть более элегантное решение - предлагайте.

  3. УдалитьНастройку - ошибка с именами переменных. Модуль не компилируется.

@240596448
Copy link
Contributor Author

240596448 commented Oct 5, 2023

  1. Если писать секцию bak. безусловно, то при записи пустого значения (старое не было установлено) - код возврата 1 (падаем). Т.е. команда совпадает с п.2.
  2. bak. пишем только при необходимости. При удалении исключаем код 128 (удаление несуществующей секции)

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Лучше заменить предыдущее условие на проверку с ИсключениеКодВозврата + обновить сообщение в логгере. Сейчас успешный случай проглатывается без следа.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ну и поменять на что-то в духе "ДопустимыйКодВозврата".

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

забирай

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ай хорошо...

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch

@nixel2007 nixel2007 merged commit a2a7c07 into nixel2007:develop Oct 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants