Skip to content

Conversation

@MaxTube-dot
Copy link
Contributor

Is your pull request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

Describe the solution you've done
Key points of your tech and arch decisions.

Related Issues
If it fixes an open issue, please link to the issue here.

@Stepami Stepami linked an issue Jan 25, 2026 that may be closed by this pull request
@MaxTube-dot
Copy link
Contributor Author

Правильно ли я понял, что ты с помощью 'RestoreEnablePackagePruning' отключил ошибку сборки и насильно подтянул . Насколько я понимаю сейчас с моими изменениями ничего измениться не должно, у меня вроде ничего не изменилось(версии пакетов), просто сделали это более элегантно. Уязвимостей тоже нет 'dotnet list package --vulnerable'

@Stepami
Copy link
Owner

Stepami commented Jan 27, 2026

@MaxTube-dot Да, я поднимал версии транзитивных зависимостей через прямые ссылки, чтобы фиксить уязвимости.

В 10ке сделали ворнинг, если пакет по прямой ссылке не используется. У меня стоит TreatWarningsAsErrors true => неиспользуемые пакеты это ошибка билда

@Stepami
Copy link
Owner

Stepami commented Jan 27, 2026

@MaxTube-dot подними ещё версии пакетов до последних актуальных пж

@MaxTube-dot
Copy link
Contributor Author

На будущее не рассматривал внедрение "Диапазона версий"? Конечно головной боли будет больше, но на каждую минорную версию комит не придется делать. Хотя и контроля меньше, но как-то от этого неспокойнее.

@Stepami
Copy link
Owner

Stepami commented Jan 27, 2026

Нет, на текущей работе нахавался диапазонов, ну нафиг.

Copy link
Owner

@Stepami Stepami left a comment

Choose a reason for hiding this comment

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

Последние штрихи

@MaxTube-dot
Copy link
Contributor Author

Готово

@Stepami Stepami self-requested a review January 27, 2026 16:31
Copy link
Owner

@Stepami Stepami left a comment

Choose a reason for hiding this comment

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

Спасибо большое! Супер, теперь красота!

@Stepami Stepami merged commit 791cc08 into Stepami:master Jan 27, 2026
0 of 2 checks passed
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.

[tech debt] CPM transitive pinning

2 participants