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

Copy new localizable attribute #104

Closed
wants to merge 2 commits into from
Closed

Conversation

Runhard13
Copy link
Contributor

Скопировал LocalizableEnumMemberAttribute.cs в Abstractions. Сделал это потому что задачу перевода всех библиотек и репозиториев на один атрибут хочу сделать так:

  1. Скопировать LocalizableEnumMemberAttribute.cs в Abstractions
  2. Заменить в проектах использование старого атрибута и нового атрибута, но из Templates на атрибут из Abstractions
  3. Удалить старый атрибут и новый атрибут в Templates

Мне кажется что при таком алгоритме ничего не сломается и всё будет работать даже тогда, когда процесс миграции атрибута со старого на новый еще не завершен

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 57% 48% 84
Mindbox.I18n.Analyzers 61% 55% 153
Mindbox.I18n.AspNetCore 86% 94% 21
Mindbox.I18n.Template 48% 59% 48
Summary 54% (463 / 856) 53% (149 / 282) 408

@Runhard13 Runhard13 closed this Dec 10, 2024
@Runhard13
Copy link
Contributor Author

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

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