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

move LocalizableEnumMemberAttribute.cs to Abstractions + delete LocalizableDisplayAttribute.cs #103

Closed
wants to merge 1 commit into from

Conversation

Runhard13
Copy link
Contributor

перенес новый атрибут в абстракции, удалил старый

delete LocalizableDisplayAttribute.cs
Copy link

github-actions bot commented Dec 9, 2024

Code Coverage

Package Line Rate Branch Rate Complexity Health
Mindbox.I18n 36% 38% 102
Mindbox.I18n.Abstractions 63% 48% 80
Mindbox.I18n.Analyzers 61% 55% 153
Mindbox.I18n.AspNetCore 86% 94% 21
Mindbox.I18n.Template 49% 59% 46
Summary 55% (463 / 844) 53% (149 / 282) 402

@Runhard13 Runhard13 closed this Dec 9, 2024
@Runhard13 Runhard13 deleted the localizable-enum-attr-movement branch December 9, 2024 16:12
@Runhard13
Copy link
Contributor Author

Mindbox.Framework обновляется довольно часто и используется много где. Поэтому я не могу сначала удалить атрибут, а потом переводить постепенно все репозитории на новую версию библиотеки

Мне сначала нужно полностью убрать из кода все места, где используется старый атрибут, а потом уже удалить его из библиотеки. Поэтому PR пока что закрываю, вернусь к нему позже

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.

1 participant