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

Перевести проект на Kotlin #81

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

Перевести проект на Kotlin #81

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

Comments

@dmitry-weirdo
Copy link
Collaborator

dmitry-weirdo commented Nov 5, 2024 •
edited
Loading

  • В целом Kotlin и Java могут сосуществовать в одном проекте/модуле, но дьявол, как всегда, в деталях.
  • Lombok будет не нужен, есть data class. При совсем плохом кейсе можно вроде бы юзать record классы из новых версий Джавы.
  • MapStruct теоретически должен работать с Котлиновскими классами, но тут возможны проблемы.
  • Генерация из OpenApi вряд ли будет работать на Kotlin.
  • Пример перехода можно посмотреть в моём другом проекте — https://github.com/dmitry-weirdo/wiki-telegram-bot.
  • Начинаем пока с модуля kgstatsSrv. Модуль kgparserSrv является более древним, его можно перевести во вторую очередь.
dmitry-weirdo added a commit that referenced this issue Nov 21, 2024
@dmitry-weirdo
feat: #81 Start including the latest kotling in the pom.xml or kgstat…
fb8395b 
…sSrv.
dmitry-weirdo added a commit that referenced this issue Nov 21, 2024
@dmitry-weirdo
feat: #81 Convert StatisticsGeneratorConfig to a simple Kotlin data c…
4922d7d 
…lass. Statistics generation is working locally.
dmitry-weirdo added a commit that referenced this issue Nov 21, 2024
@dmitry-weirdo
feat: #81 Migrate StatisticsGenerator to Kotlin.
fbd0e59
dmitry-weirdo added a commit that referenced this issue Nov 21, 2024
@dmitry-weirdo
feat: #81 Migrate ExportContext to Kotlin. Use dokka plugin instead o…
70b2799 
…f maven-javadoc-plugin to support Kotlin classes and not fail.
dmitry-weirdo added a commit that referenced this issue Nov 21, 2024
@dmitry-weirdo
feat: #81 Some re-build magic - actually the Lombok-generated getters…
6c83724 
… ARE working in Kotlin classes (even in IntelliJ IDEA).
dmitry-weirdo added a commit that referenced this issue Nov 22, 2024
@dmitry-weirdo
refactor: #81 In PageUrls, extract the common path pattern to a commo…
31f308e 
…n method.
dmitry-weirdo added a commit that referenced this issue Nov 22, 2024
@dmitry-weirdo
feat: #81 Migrate VocabularyTopUtils and ExporterUtils to Kotlin.
e2e0af8
dmitry-weirdo added a commit that referenced this issue Nov 22, 2024
@dmitry-weirdo
feat: #81 Migrate ZipUtils to Kotlin. Close the ZipFile resource corr…
2e76770 
…ectly.
dmitry-weirdo added a commit that referenced this issue Nov 22, 2024
@dmitry-weirdo
chore: #81 Make fields final when they can be final (found by the…
7aa1fdd 
… corresponding inspection).
dmitry-weirdo added a commit that referenced this issue Nov 22, 2024
@dmitry-weirdo
feat: #81 Converted PageUrls to Kotlin. Looks less cumbersome now.
a835e04
dmitry-weirdo added a commit that referenced this issue Nov 22, 2024
@dmitry-weirdo
feat: #81 Convert HaulUtils and HaulUtilsTest to Kotlin.
38e9f67
dmitry-weirdo added a commit that referenced this issue Nov 22, 2024
@dmitry-weirdo
refactor: #81 In StandardDictionary enum, use enum fields instead of …
7cfedbf 
…ugly static methods (same as it was done for the Rank enum).
dmitry-weirdo added a commit that referenced this issue Nov 25, 2024
@dmitry-weirdo
feat: #81 #102 Convert Car to Kotlin. Move CarTest to kg-common.
61a3fec
dmitry-weirdo added a commit that referenced this issue Nov 25, 2024
@dmitry-weirdo
test: #81 Fix JacksonUtilsTest: access kotlin Car enum fields via…
5d89dec 
… getters.
dmitry-weirdo added a commit that referenced this issue Nov 25, 2024
@dmitry-weirdo
feat: #81 #102 Move Wiki and StandardDictionary to kg-common an…
7f249c4 
…d convert them to Kotlin.
dmitry-weirdo added a commit that referenced this issue Nov 25, 2024
@dmitry-weirdo
refactor: #81 In NonStandardDictionary, use displayName and `disp…
53c374d 
…layNamePrepositional` field names, same as in `StandardDictionary`.
dmitry-weirdo added a commit that referenced this issue Nov 25, 2024
@dmitry-weirdo
feat: #81 Move UrlConstructor to kg-common and convert it to Kotl…
e732c73 
…in. Extract static methods from Dictionary to a common Kotlin clas `DictionaryUtils`.
dmitry-weirdo added a commit that referenced this issue Nov 25, 2024
@dmitry-weirdo
feat: #81 #102 Move Dictionary and StandardDictionary to `kg-comm…
607cd33 
…on` and convert them to Kotlin. Make fields of `Car` visible to Java.
dmitry-weirdo added a commit that referenced this issue Nov 25, 2024
@dmitry-weirdo
feat: #81 #102 Move NonStandardDictionaryType to kg-common and co…
8239c58 
…nvert it to Kotlin.
dmitry-weirdo added a commit that referenced this issue Nov 25, 2024
@dmitry-weirdo
feat: #81 #102 Move NonStandardDictionary to kg-common and conver…
013f49c 
…t it to Kotlin.
dmitry-weirdo added a commit that referenced this issue Nov 25, 2024
@dmitry-weirdo
feat: #81 #102 Move Rank to kg-common and convert it to Kotlin. N…
bbea121 
…o more dependencies from `kgstats-srv` to `kgparser-srv`!!!
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.
@dmitry-weirdo dmitry-weirdo reopened this Nov 26, 2024
@dmitry-weirdo
Copy link
Collaborator Author

Переведен не весь проект, а только часть.

@dmitry-weirdo dmitry-weirdo reopened this Nov 26, 2024
dmitry-weirdo added a commit that referenced this issue Nov 26, 2024
@dmitry-weirdo
feat: #81 Add the method for generating "get user book stats" API URL…
c84b2f5 
… (copy-pasted from "CSV files" branch and moved into Kotlin).
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