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.

Sign up for GitHub

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

Jump to bottom

Убрать использование легаси-библиотеки Kefir #101

Open
dmitry-weirdo opened this issue Nov 22, 2024 · 1 comment
Open

Убрать использование легаси-библиотеки Kefir #101

dmitry-weirdo opened this issue Nov 22, 2024 · 1 comment

Comments

@dmitry-weirdo
Copy link
Collaborator

  • Использования должны находиться только в легаси-модулях kgparser-srv и kgparser-web.
  • StringUtils.concat можно заменить на String.format (для Java) или Kotlin string template.
  • ❓ Насчёт остальных использований нужно думать, мб сложнее.
dmitry-weirdo added a commit that referenced this issue Nov 22, 2024
@dmitry-weirdo
feat: #101 Reduce kefir's StringUtils.concat usage - use String.forma…
b22af04 
…t instead.
dmitry-weirdo added a commit that referenced this issue Nov 22, 2024
@dmitry-weirdo
chore: #101 Remove kefir's StringUtils.concat usage from kgparser-srv…
7f4a4df 
… module.
dmitry-weirdo added a commit that referenced this issue Nov 25, 2024
@dmitry-weirdo
feat: #101 Remove kefir's StringUtils.concat usage from kgparser-web …
10baa8b 
…module.
@dmitry-weirdo
Copy link
Collaborator Author

dmitry-weirdo commented Nov 25, 2024 •
edited
Loading

  • Использование StringUtils.concat убрал.
  • Согласно Выделить из kgparserSrv модуль kg-common, в котором будут базовые классы, используемые и легаси-парсером соревнований, и статистикой #102, теперь kefir используется только в легаси-модулях kgparserSrv и kgparserWeb. Эти модули теперь по дефолту выключены.
  • Использование всяких InitiableAction и JsonServlet будет убрать сложнее — нужно просто эти модули перевести с кефирного извращения на Spring Boot, вместо JsonObject (идущего своими корнями к org.json) там будет автоматически использоваться нормальный jackson.
  • Таким образом, эту задачу можно закрывать.
    • И можно открыть отдельную супер-низко-приоритетную задачу по миграции старых модулей на Spring Boot.

dmitry-weirdo added a commit that referenced this issue Nov 26, 2024
@dmitry-weirdo
Merge pull request #100 from kghackers/feat/#81_migrate-to-kotlin
5ee1ce9 
feat: #81 Migrate project to Kotlin (started the migration)
feat: #102 Extract `kgCommon` module, turn off the legacy modules by default (you can enable them via a special maven profile).
feat: #101 Kefir usage reduced, only left in the legacy modules.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@dmitry-weirdo