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

Issues 155, 173 #164

Merged
merged 20 commits into from
Aug 19, 2019
Merged

Issues 155, 173 #164

merged 20 commits into from
Aug 19, 2019

Conversation

sharkovadarya
Copy link
Contributor

@sharkovadarya sharkovadarya commented Aug 1, 2019

ruleorder, localrules should resolve/complete not only rules from same file, but all rules from project #155

I assumed resolve/completion should work not for all files in the project, but only for those included in the current file via include in this file or other files included in this file.

show all names only on 2nd auto-completion invocaton #173

Darya.Sharkova added 13 commits August 14, 2019 10:37
# Conflicts:
#	src/main/kotlin/com/jetbrains/snakecharm/lang/psi/impl/SmkReferenceExpressionImpl.kt
#	src/main/kotlin/com/jetbrains/snakecharm/lang/psi/impl/SmkWorkflowArgsSectionImpl.kt
#	src/main/kotlin/com/jetbrains/snakecharm/lang/psi/types/AbstractSmkRuleOrCheckpointType.kt
…or all indexed elements, completion works for local files on 1st invocation, all files on further invocations.
…est for multiple autocompletion invocations.
…les if at least one reference resolves to a rule from a current/included file.
…light rules if at least one reference resolves to a rule from a current/included file."

This reverts commit 828750c.
…uded/present in the current file."

This reverts commit 3a650a3.
…sent; modified rules/checkpoints completion to use invocation count.
Darya.Sharkova added 3 commits August 19, 2019 15:06
@iromeo iromeo changed the title Issue 155 Issues 155, 173 Aug 19, 2019
@iromeo iromeo merged commit d49a487 into master Aug 19, 2019
@iromeo iromeo deleted the 155 branch July 2, 2020 16:33
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