Skip to content
This repository was archived by the owner on Apr 14, 2022. It is now read-only.

Commit ade27a4

Browse files
author
MikhailArkhipov
committed
Merge fixes from release
2 parents 6daa3ac + 8e91037 commit ade27a4

4 files changed

Lines changed: 61 additions & 9 deletions

File tree

src/LanguageServer/Impl/Implementation/Server.cs

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -381,11 +381,7 @@ private async Task DoInitializeAsync(InitializeParams @params, CancellationToken
381381
}
382382
_displayTextBuilder = DocumentationBuilder.Create(DisplayOptions);
383383

384-
if (string.IsNullOrEmpty(Analyzer.InterpreterFactory?.Configuration?.InterpreterPath)) {
385-
LogMessage(MessageType._General, "Initializing for generic interpreter");
386-
} else {
387-
LogMessage(MessageType._General, $"Initializing for {Analyzer.InterpreterFactory.Configuration.InterpreterPath}");
388-
}
384+
DisplayStartupInfo();
389385

390386
if (@params.rootUri != null) {
391387
_rootDir = @params.rootUri.ToAbsolutePath();
@@ -399,6 +395,14 @@ private async Task DoInitializeAsync(InitializeParams @params, CancellationToken
399395
Analyzer.Interpreter.ModuleNamesChanged += Interpreter_ModuleNamesChanged;
400396
}
401397

398+
private void DisplayStartupInfo() {
399+
LogMessage(MessageType._General, Resources.LanguageServerVersion.FormatInvariant(Assembly.GetExecutingAssembly().GetName().Version));
400+
LogMessage(MessageType._General,
401+
string.IsNullOrEmpty(Analyzer.InterpreterFactory?.Configuration?.InterpreterPath)
402+
? Resources.InitializingForGenericInterpreter
403+
: Resources.InitializingForPythonInterpreter.FormatInvariant(Analyzer.InterpreterFactory.Configuration.InterpreterPath));
404+
}
405+
402406
private void Interpreter_ModuleNamesChanged(object sender, EventArgs e) {
403407
Analyzer.Modules.ReInit();
404408
foreach (var entry in Analyzer.ModulesByFilename) {
@@ -437,7 +441,7 @@ private async Task<PythonAnalyzer> CreateAnalyzer(PythonInitializationOptions.In
437441

438442
var interp = factory.CreateInterpreter();
439443
if (interp == null) {
440-
throw new InvalidOperationException("Failed to create interpreter");
444+
throw new InvalidOperationException(Resources.Error_FailedToCreateInterpreter);
441445
}
442446

443447
LogMessage(MessageType.Info, $"Created {interp.GetType().FullName} instance from {factory.GetType().FullName}");

src/LanguageServer/Impl/PathsWatcher.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ internal sealed class PathsWatcher : IDisposable {
3333

3434
public PathsWatcher(string[] paths, Action onChanged, ILogger log) {
3535
_log = log;
36-
paths = paths.Where(p => Path.IsPathRooted(p)).ToArray() ?? Array.Empty<string>();
36+
paths = paths != null ? paths.Where(p => Path.IsPathRooted(p)).ToArray() : Array.Empty<string>();
3737
if (paths.Length == 0) {
3838
return;
3939
}
@@ -46,7 +46,7 @@ public PathsWatcher(string[] paths, Action onChanged, ILogger log) {
4646
if (!Directory.Exists(p)) {
4747
continue;
4848
}
49-
} catch(IOException ex) {
49+
} catch (IOException ex) {
5050
_log.TraceMessage($"Unable to access directory {p}, exception {ex.Message}");
5151
continue;
5252
}
@@ -66,7 +66,7 @@ public PathsWatcher(string[] paths, Action onChanged, ILogger log) {
6666
.Add(() => fsw.Deleted -= OnChanged)
6767
.Add(() => fsw.EnableRaisingEvents = false)
6868
.Add(fsw);
69-
} catch(ArgumentException ex) {
69+
} catch (ArgumentException ex) {
7070
_log.TraceMessage($"Unable to create file watcher for {p}, exception {ex.Message}");
7171
}
7272
}

src/LanguageServer/Impl/Resources.Designer.cs

Lines changed: 36 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/LanguageServer/Impl/Resources.resx

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -123,6 +123,18 @@
123123
<data name="AnalysisProgress_SingleItemRemaining" xml:space="preserve">
124124
<value>Analyzing workspace, 1 item remaining...</value>
125125
</data>
126+
<data name="Error_FailedToCreateInterpreter" xml:space="preserve">
127+
<value>Failed to create interpreter</value>
128+
</data>
129+
<data name="InitializingForGenericInterpreter" xml:space="preserve">
130+
<value>Initializing for generic interpreter</value>
131+
</data>
132+
<data name="InitializingForPythonInterpreter" xml:space="preserve">
133+
<value>Initializing for {0}</value>
134+
</data>
135+
<data name="LanguageServerVersion" xml:space="preserve">
136+
<value>Microsoft Python Language Server version {0}</value>
137+
</data>
126138
<data name="RenameVariable_CannotRename" xml:space="preserve">
127139
<value>Cannot rename</value>
128140
</data>

0 commit comments

Comments
 (0)