Allow duplicate LSP completions for autoimport

[jeremija]

Similarly to TS completions, this allows the user to pick the package to autoimport when both the package and method name are the same.

See the screenshot below for example:

Previously only a single errors.New method would be visible.

[stale]

This pull request has been automatically marked as stale because it has not been updated recently. Make sure to write tests and document your changes. See :help ale-dev for information on writing tests.
If your pull request is good to merge, bother w0rp or another maintainer again, and get them to merge it.

[jeremija]

Hi @w0rp - happy New Year! Sorry to bother you, this was just automatically market as stale, but I'd like to have this merged since I use this a lot. Is there something that needs fixing before it's merged?

[hsanson]

ALE migrated from Travis to Github Actions (see #3548) so it is necessary to rebase this PR from latest master branch to trigger the required checks.

# Add dense-analysis as remote upstream on your local fork. Not 
# needed if already added.
git remote add upstream https://github.com/dense-analysis/ale.git

# Sync your local master with upstream master
git checkout master
git fetch upstream master
git merge upstream/master

# Rebase the PR branch with master
git checkout my-branch-name
git rebase master   # Fix any conflicts you may have

# Force push the updated PR branch to origin to trigger the checks
git push -f origin my-branch-name

[jeremija]

@hsanson thanks for the info - just rebased and all the tests have passed. Are there any blockers for merging this? It was a pretty simple change.

[w0rp]

Looks good to me.

[w0rp]

Cheers! 🍻

[jeremija]

Thank you!