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

Refactoring/upgrade framework version 6lts #37

Conversation

sfsharapov
Copy link

По задаче #36 переход на новую версию каркаса (фреймворка).

Тесты нужно полностью переработать. Поработал над совместимостью с новым интерфейсом библиотеки EasyNetQ

x => x.Bind(It.IsAny<IExchange>(), It.IsAny<IQueue>(), It.Is<string>(key => key == routings[1])),
Times.AtLeastOnce);
}
// [TestCase("", "", false)]
Copy link
Member

Choose a reason for hiding this comment

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

закомментированные тесты - не вариант, нужно восстановить их работоспособность

@PavelKulbida
Copy link
Member

то что переведено на .NET6 - это замечательно !
но похоже что также необходимо скорректировать pipeline сборки, сейчас он завален

@PavelKulbida
Copy link
Member

PavelKulbida commented Dec 19, 2021

Тесты нужно полностью переработать

а по какой причине ?

@RomanProsin
Copy link
Member

RomanProsin commented Dec 19, 2021

Добрый день. Спасибо за инициативу. Мы рады вашему желанию внести свой вклад в проект. Однако данный PR не можем принять. Проект временно заморожен, с целью проведения редизайна. Это изменение вносит неразрешимые конфликты, так как оригинальный репозиторий ушёл далеко вперёд. Кроме того, ещё очень рано для перевода на .NET 6, необходимо выждать время (для обнаружения уязвимостей и серьёзных дефектов и их исправления). Кодовая база сканируется инструментом Checkmarx, необходимо дождаться полной поддержки этим инструментом 6 версии платформы. В первом квартале 2022 года проект будет разморожен все изменения будут добавлены из основного репозитория проекта.

@RomanProsin RomanProsin linked an issue Dec 19, 2021 that may be closed by this pull request
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.

Переход на .NET 6
3 participants