Skip to content

Conversation

@partizand
Copy link

Позволяет включать выгрузку в edt по желанию.
Возможно решит проблему #43
Нарушает обратную совместимость.

@partizand partizand changed the title Добавлен фалаг включения плагина #43 Добавлен фалаг включения плагина Dec 28, 2025
@github-actions
Copy link

github-actions bot commented Dec 28, 2025

TestsPassed ☑️SkippedFailed ❌️
Результаты тестов. ОС: windows-latest. Версия 1С: 8.3.24.1691. Версия OneScript: 1.9.2. Версия EDT: 2024.2.5255 ran252 passed0 skipped3 failed

@github-actions
Copy link

github-actions bot commented Dec 28, 2025

TestsPassed ☑️SkippedFailed ❌️
Результаты тестов. ОС: ubuntu-22.04. Версия 1С: 8.3.24.1691. Версия OneScript: 1.9.2. Версия EDT: 2024.2.5255 ran252 passed0 skipped3 failed

@github-actions
Copy link

github-actions bot commented Dec 28, 2025

TestsPassed ☑️SkippedFailed ❌️
Результаты тестов. ОС: windows-latest. Версия 1С: 8.3.24.1691. Версия OneScript: 1.9.2. Версия EDT: 2023.3.6255 ran252 passed0 skipped3 failed

@github-actions
Copy link

github-actions bot commented Dec 28, 2025

TestsPassed ☑️SkippedFailed ❌️
Результаты тестов. ОС: windows-latest. Версия 1С: 8.3.21.1624. Версия OneScript: 1.9.2. Версия EDT: 2023.3.6255 ran252 passed0 skipped3 failed

@github-actions
Copy link

github-actions bot commented Dec 28, 2025

TestsPassed ☑️SkippedFailed ❌️
Результаты тестов. ОС: windows-latest. Версия 1С: 8.3.21.1624. Версия OneScript: 1.9.2. Версия EDT: 2024.2.5255 ran252 passed0 skipped3 failed

@github-actions
Copy link

github-actions bot commented Dec 28, 2025

TestsPassed ☑️SkippedFailed ❌️
Результаты тестов. ОС: ubuntu-22.04. Версия 1С: 8.3.24.1691. Версия OneScript: 1.9.2. Версия EDT: 2023.3.6255 ran252 passed0 skipped3 failed

@github-actions
Copy link

github-actions bot commented Dec 28, 2025

TestsPassed ☑️SkippedFailed ❌️
Результаты тестов. ОС: ubuntu-22.04. Версия 1С: 8.3.21.1624. Версия OneScript: 1.9.2. Версия EDT: 2023.3.6255 ran252 passed0 skipped3 failed

@github-actions
Copy link

github-actions bot commented Dec 28, 2025

TestsPassed ☑️SkippedFailed ❌️
Результаты тестов. ОС: ubuntu-22.04. Версия 1С: 8.3.21.1624. Версия OneScript: 1.9.2. Версия EDT: 2024.2.5255 ran252 passed0 skipped3 failed

@ovcharenko-di
Copy link
Collaborator

спасибо за PR!

У меня есть несколько вопросов:

  1. Зачем потребовался флаг, если можно выполнить просто:
gitsync plugins disable edtExport
gitsync sync

Если выяснится, что флаг действительно нужен, то надо рассмотреть еще несколько вопросов:

  1. Если правильно понимаю, то название флага --edt-format не соответствует поведению. Это название не подразумевает какого-то "включения-выключения".
  2. Тесты упали, надо разобраться и починить. Возможно, это не связано напрямую с текущим PR.
  3. В тестах нет проверки поведения с разным значением флага.

@partizand
Copy link
Author

1. Зачем потребовался флаг, если можно выполнить просто:
gitsync plugins disable edtExport
gitsync sync

Можно. Но мне кажется странным необходимость отключения плагина, что бы вернуть исходную функциональность. Кажется установка плагина не должна менять поведение.
Кроме того это убирает ошибку init. Но убирать её конечно лучше по другому.

2. Если правильно понимаю, то название флага --edt-format не соответствует поведению. Это название не подразумевает какого-то "включения-выключения".

Можно поменять на любое, это не принципиально. Название флага начинается на имя плагина. А то уже тесно с этими флагами.

3. Тесты упали, надо разобраться и починить. Возможно, это не связано напрямую с текущим PR.

Немного глянул. Не пойму, как связано с изменениями. Неудобно, что тестирование требует админских прав. Да и тестирование мне сложно пока освоить.

4. В тестах нет проверки поведения с разным значением флага.

Посмотрю.

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