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

Order direction keywords are not suggested in order-clause context #33895

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

Order direction keywords are not suggested in order-clause context #33895

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

Comments

@malinthar
Copy link
Contributor

Description:

function findToptenCountriesByAthletes() returns error? {
    map<int> athletesByCountry = check findNumOfAthletesBycountry();
    var result = from [string,int] entry in athletesByCountry.entries() 
                     order by entry[1] <cursor>
}

The completion list at the depicted cursor position does not contain the ascending and descending keywords.
This issue seems to be with the resolved completion context.

Steps to reproduce:
Mentioned above

@malinthar malinthar added Type/Bug Team/LanguageServer Language Server Implementation related issues. #Compiler Area/Completion Issues related to Language Server Auto Completions labels Nov 18, 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 Priority/High Team/LanguageServer Language Server Implementation related issues. #Compiler Type/Bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants