GeezerVisonThe World Wide Web, restored to 800 pixels of glory. |
| Home|About|Gaming cabinet|Archives|How it works|Starter sites |
GeezerVison is showing github.com — Messed plugin config list order · Issue #677 · python-lsp/python-lsp-server Open DescriptionHi! TL;DR I think that merge_dicts implementation in def merge_dicts(dict_a, dict_b): """Recursively merge dictionary b into dictionary a. If override_nones is True, then """ def _merge_dicts_(a, b): for key in set(a.keys()).union(b.keys()): if key in a and key in b: if isinstance(a[key], dict) and isinstance(b[key], dict): yield (key, dict(_merge_dicts_(a[key], b[key]))) elif isinstance(a[key], list) and isinstance(b[key], list): yield (key, list(set(a[key] + b[key]))) # <--- this line here is wrong elif b[key] is not None: yield (key, b[key]) else: yield (key, a[key]) elif key in a: yield (key, a[key]) elif b[key] is not None: yield (key, b[key]) return dict(_merge_dicts_(dict_a, dict_b)) By converting the list to a set, it not only gets rid of the duplicates, but also messes up the list order unpredictably. Replacing I'm using pylsp v1.13.1 with pylsp_mypy v0.7.0. (The whole LSP is run by Emacs v30.1 via eglot, but this probably doesn't matter.) And I noticed the following strange behavior, which I can't really put my finger on yet. When pylsp invokes pylsp_mypy, it uses the following configuration: Note the Note how the I also noticed that it's not always reverse, the order seems to be pretty random. Sometimes it's ok and works, other times, it's messed up. Reacted by George Kettleborough Reacted by Robert Timm Activity
MetadataMetadataAssigneesLabelsNo labels TypeProjectsMilestoneRelationshipsDevelopmentIssue actions |
| You are visitor 01402503 — GeezerVison does not store the pages you look up. Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required. |