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

Completion base support #22

Merged
merged 8 commits into from
Nov 25, 2019
Merged

Completion base support #22

merged 8 commits into from
Nov 25, 2019

Conversation

ileasile
Copy link
Contributor

Completion works on TAB.
The following tokens are took int account:

  • user-defined variables
  • user-defined functions
  • keywords

build.gradle Outdated
@@ -23,6 +23,7 @@ allprojects {
jcenter()
mavenLocal()
mavenCentral()
maven { url '/home/ileasile/maven_repo/repo_1/maven' }
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, remove this

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed

this.codeOnLoad = ArrayList()
}

fun receiver(receiver: KotlinReceiver): KotlinReplProperties {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is no need in setter methods, all fields in Kotlin are properties and optional getters/setters can be added if needed (https://kotlinlang.org/docs/reference/properties.html)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I know this, but this was auto-converted from Java :)

val receivers = listOf(kt)
implicitReceivers.invoke(receivers)

Unit
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need to return Unit here

@nikitinas nikitinas merged commit 1672771 into nikitin Nov 25, 2019
@ileasile ileasile deleted the completion branch December 27, 2019 11:02
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.

2 participants