-
Notifications
You must be signed in to change notification settings - Fork 20
Отладочное приложение и работающий netcoreapp3.1 #75
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
base: develop
Are you sure you want to change the base?
Отладочное приложение и работающий netcoreapp3.1 #75
Conversation
* также добавлено отладочное приложение, для проверки поведения в режиме реального времени
* статических файлов * фоновых заданий под 3.1
* используется устаревший механизм формирования представлений
|
Занятно. Но если я правильно понимаю, что мерж этого PR не имеет смысла, пока доработки движка (которые как раз и демонстрирует этот мерж) не будут влиты в паблик, все верно? |
|
@EvilBeaver скорее всего - ASP 3.1 ведет себя координально по другому. Починить тесты сходу не удалось. |
Ради такого дела, тесты можно и не чинить, я и сам бы починил ;) Просто было бы странно влить PR с демо-приложением, которое заведомо не работает не только из-за тестов, но и из-за в принципе отсутствия поддерживающего кода на стороне движка |
|
@EvilBeaver погоди - демоприложение работает. Мы просто не стали в этот PR вливать фронт - слишком много изменений |
| provider?.Configure(services); | ||
|
|
||
| /* | ||
| app.UseEndpoints(endpoints => |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Этот камент тогда не нужен, раз мы его выключаем?
| using System.Linq; | ||
| using Microsoft.AspNetCore.Mvc.ViewComponents; | ||
| using Microsoft.AspNetCore.Mvc.ViewFeatures.Internal; | ||
| //using Microsoft.AspNetCore.Mvc.ViewFeatures.Internal; ASP.NET Breaking Changes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Если интернала больше нет, что вместо него? Если все перенесли в паблик, тогда эту строчку надо удалить, а не закомментировать.
| using Microsoft.AspNetCore.Mvc.ApplicationModels; | ||
| using Microsoft.AspNetCore.Mvc.Authorization; | ||
| using Microsoft.AspNetCore.Mvc.Internal; | ||
| //using Microsoft.AspNetCore.Mvc.Internal; ASP.NET Breaking Changes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Если интернала больше нет, что вместо него? Если все перенесли в паблик, тогда эту строчку надо удалить, а не закомментировать.
| using System.Linq; | ||
| using System.Reflection; | ||
| using Microsoft.AspNetCore.Hosting.Internal; | ||
| //using Microsoft.AspNetCore.Hosting.Internal; ASP.NET Breaking Changes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Если интернала больше нет, что вместо него? Если все перенесли в паблик, тогда эту строчку надо удалить, а не закомментировать.
даже со старыми пакетами - старая функциональность работает, но уже с новым
netcoreapp3.1, тесты естественно не пройдут - использовался Internal APi котороенельзя-ай-ай-яйдля отладки добавлен
dotnet watchчтобы можно было в онлайне перекомпилировать код и смотреть поведение без Visual Studioиз изменений
endpointsloggerдля Hangfiredotnet watchcsprojдобавлена функция слежения заpackage-loaderизменеями и за всем каталогом - за C# файлам следит автоматическипакеты не обновлялись