fix: set score completion item when skipfilter true

[akoreman]

Issue #, if available: NA

Description of changes: When we set skipFilter for a completion item to true, the filtering logic skips the part where $score is set which results in the score property of the item being ignored, resulting in unexpected ordering of items.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Pull Request Checklist:

• No backwards incompatible changes were made to Ace's public interface which is defined through the main typings file (ace.d.ts) and its references:
  • https://github.com/ajaxorg/ace/blob/master/ace.d.ts
  • https://github.com/ajaxorg/ace/blob/master/ace-modes.d.ts
  • https://github.com/ajaxorg/ace/blob/master/ace-extensions.d.ts

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.08%. Comparing base (fbe3ddc) to head (0a0df17).
Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #5749   +/-   ##
=======================================
  Coverage   87.08%   87.08%           
=======================================
  Files         598      598           
  Lines       43801    43802    +1     
  Branches     7171     7171           
=======================================
+ Hits        38143    38144    +1     
  Misses       5658     5658           

Flag	Coverage Δ
unittests	87.08% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.