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

Kotlin Rewrite #123

Closed
lokiuox opened this issue Jul 27, 2023 · 1 comment
Closed

Kotlin Rewrite #123

lokiuox opened this issue Jul 27, 2023 · 1 comment
Assignees
Labels
Effort: 1 week The implementation should require about one week Enhancement New feature or request
Milestone

Comments

@lokiuox
Copy link

lokiuox commented Jul 27, 2023

The project's reliance on Jython makes it stuck on old technologies and difficult to maintain and improve. As the project grows bigger, the natural next step is moving toward a more modern language that will make the code structure more robust and organized. This will make it easier to understand and contribute to. This rewrite will be prioritized in order to get rid of the technical debt introduced by Jython as soon as possible and allow new features and bugfixes to be implemented directly in the new Kotlin version.

The initial goal will be to port everything that is currently implemented in Python except for the gqlspection library, for which we will introduce an interface that will allow it to be seamlessly replaced with a Java/Kotlin implementation at a later time. The rest of the code will be ported to Kotlin in almost a 1:1 way at first, but reorganizing/restructuring it where needed to be more in line with typical OO design patterns and generally more readable. Some of the planned new features may also be directly implemented during the rewrite if time permits.

@lokiuox lokiuox added the Enhancement New feature or request label Jul 27, 2023
@lokiuox lokiuox added this to the v5.1 milestone Jul 27, 2023
@lokiuox lokiuox self-assigned this Jul 27, 2023
@lokiuox lokiuox added the Effort: 1 week The implementation should require about one week label Jul 27, 2023
@execveat
Copy link
Contributor

The rewrite has been pushed to dev branch. Thank you @lokiuox!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Effort: 1 week The implementation should require about one week Enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants