Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Обсуждение. #106

Open
VirRus77 opened this issue Nov 30, 2022 · 3 comments
Open

Обсуждение. #106

VirRus77 opened this issue Nov 30, 2022 · 3 comments

Comments

@VirRus77
Copy link
Contributor

Приветствую.
Посмотрел код. Коду требуется глобальный рефакторинг.
Особенно в выделении из UI логической части.
Вот думаю, тебе как автору это требуется или нет?
Просто изменений за один раз будет много, очень много....
П.С. просто делать глобальный рефакторинг с пул реквестом или нет?
П.С.2 И добавь пожалуйста UnityModManagerConfig.xml в контроль версий =)

@newman55
Copy link
Owner

newman55 commented Dec 1, 2022

Привет.

Я наверно скорее против, привык как есть, да и все публичные функции должны быть сохранены, что уменьшает возможности рефакторинга.

Этот файл я не добавляю из-за того что он указывает на их взлом.

@newman55
Copy link
Owner

newman55 commented Dec 8, 2022

В версии 0.25.1 файлы гармонии вместо net35 ошибочно попали net472. Я вернул их обратно, проверь свои моды на версии 0.25.3.

@VirRus77
Copy link
Contributor Author

Спасибо, проверил.
Мод под net47 спокойно загружается библиотекой harmony под net35.

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

No branches or pull requests

2 participants