actions/languageservices
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #165 from github/cschleiden/do-not-override-event-payloads

Browse Source 
Do not replace event payloads with null
This commit is contained in:
Christopher Schleiden
2023-02-23 14:42:02 -08:00
committed by GitHub
parent 6843c36809 89338e8f42
commit bb7fe23d4d
1 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
languageservice/src/context-providers/events/eventPayloads.ts
+6
View File
@@ -93,6 +93,12 @@ function mergeParam(target: DescriptionDictionary, param: Param) {
} else { } else {
// Otherwise add as a null value. We do not care about the actual content for validation // Otherwise add as a null value. We do not care about the actual content for validation
// auto-completion. Possible existence and the description are enough. // auto-completion. Possible existence and the description are enough.
//
// As a special case, if the param is already set, do not overwrite it.
if (!!target.get(param.name)) {
return;
}
target.add(param.name, new data.Null(), param.description); target.add(param.name, new data.Null(), param.description);
} }
} }