Skip to content

Conversation

@silverbulleters-github-service

даже со старыми пакетами - старая функциональность работает, но уже с новым netcoreapp3.1, тесты естественно не пройдут - использовался Internal APi которое нельзя-ай-ай-яй

для отладки добавлен dotnet watch чтобы можно было в онлайне перекомпилировать код и смотреть поведение без Visual Studio

из изменений

  • добавлена функция явного отключения endpoints
  • добавлено явное указание logger для Hangfire
  • добавлено приложение dotnet watch
  • в csproj добавлена функция слежения за package-loader изменеями и за всем каталогом - за C# файлам следит автоматически

пакеты не обновлялись

Alexey Lustin and others added 6 commits July 4, 2020 13:12
* также добавлено отладочное приложение,
 для проверки поведения в режиме реального времени
* статических файлов
* фоновых заданий
под 3.1
* используется устаревший механизм формирования представлений
@silverbulleters-github-service
Copy link
Author

@EvilBeaver
Copy link
Owner

@EvilBeaver https://youtu.be/4-tdrIw-lng

Занятно. Но если я правильно понимаю, что мерж этого PR не имеет смысла, пока доработки движка (которые как раз и демонстрирует этот мерж) не будут влиты в паблик, все верно?

@silverbulleters-github-service
Copy link
Author

@EvilBeaver скорее всего - ASP 3.1 ведет себя координально по другому. Починить тесты сходу не удалось.
Закроем данный PR и сделаем уже по новому.

@EvilBeaver
Copy link
Owner

EvilBeaver commented Jul 14, 2020

@EvilBeaver скорее всего - ASP 3.1 ведет себя координально по другому. Починить тесты сходу не удалось.
Закроем данный PR и сделаем уже по новому.

Ради такого дела, тесты можно и не чинить, я и сам бы починил ;) Просто было бы странно влить PR с демо-приложением, которое заведомо не работает не только из-за тестов, но и из-за в принципе отсутствия поддерживающего кода на стороне движка

@silverbulleters-github-service
Copy link
Author

@EvilBeaver погоди - демоприложение работает. Мы просто не стали в этот PR вливать фронт - слишком много изменений

provider?.Configure(services);

/*
app.UseEndpoints(endpoints =>
Copy link
Owner

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
Copy link
Owner

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
Copy link
Owner

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
Copy link
Owner

Choose a reason for hiding this comment

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

Если интернала больше нет, что вместо него? Если все перенесли в паблик, тогда эту строчку надо удалить, а не закомментировать.

@aliczin aliczin mentioned this pull request Nov 4, 2020
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