Skip to content

Commit

Permalink
Merge pull request #19537 from unoplatform/dev/jela/json-relaxed
Browse files Browse the repository at this point in the history
fix(vsix): Relax global.json parsing
  • Loading branch information
jeromelaban authored Feb 18, 2025
2 parents 0db3e8a + 6cf5044 commit 117bd4b
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/Uno.UI.RemoteControl.VS/GlobalJsonObserver.cs
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ internal class GlobalJsonObserver
private readonly Action<string> _errorAction;
private readonly InfoBarFactory _infoBarFactory;
private FileSystemWatcher? _fileWatcher;
private readonly JsonSerializerOptions _readerOptions = new() { ReadCommentHandling = JsonCommentHandling.Skip };
private readonly JsonSerializerOptions _readerOptions = new() { ReadCommentHandling = JsonCommentHandling.Skip, AllowTrailingCommas = true };

public GlobalJsonObserver(
AsyncPackage asyncPackage
Expand Down Expand Up @@ -90,7 +90,7 @@ private void ObserveChanges(string? globalJsonPath)
}
catch (Exception ex)
{
_debugAction($"GlobalJsonObserver: Error reading global.json: {ex.Message}");
_errorAction($"GlobalJsonObserver: Error reading global.json: {ex.Message}");
}
};

Expand Down

0 comments on commit 117bd4b

Please sign in to comment.