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 item sorting order in from caluse context is not accurate #33861

Closed
Tracked by #33628
malinthar opened this issue Nov 16, 2021 · 1 comment · Fixed by #34264
Closed
Tracked by #33628

Completion item sorting order in from caluse context is not accurate #33861

malinthar opened this issue Nov 16, 2021 · 1 comment · Fixed by #34264
Assignees
Labels
Area/Completion Issues related to Language Server Auto Completions Points/2 Equivalent to two days effort Team/LanguageServer Language Server Implementation related issues. #Compiler Type/Improvement

Comments

@malinthar
Copy link
Contributor

Description:
The order of completion items in the from clause context can be improved to sort the iterable expressions to the
top. The order is currently as follows.
sorting1

Steps to reproduce:

function getMedalStats() returns stream<MedalStatsRecord, error?> | error {
    int abc;
    string cd;
    int[] myArr;
    stream<string[], io:Error?>|error tsvStream = readTsvAsStream("/data/Medals.tsv");
    if readTsvAsStreamResult is stream<string[], io:Error?> {
        return stream from var entry in 
    } 
}
@malinthar malinthar added Type/Bug Team/LanguageServer Language Server Implementation related issues. #Compiler Area/Completion Issues related to Language Server Auto Completions labels Nov 16, 2021
@malinthar
Copy link
Contributor Author

Sub task of #33628

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area/Completion Issues related to Language Server Auto Completions Points/2 Equivalent to two days effort Team/LanguageServer Language Server Implementation related issues. #Compiler Type/Improvement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants