Skip to content

Enable nullable: System.Management.Automation.Language.IScriptPosition #14161

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
iSazonov merged 3 commits into from
May 18, 2021
Merged

Enable nullable: System.Management.Automation.Language.IScriptPosition #14161

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
71e3c8a
Enable nullable: System.Management.Automation.Language.IScriptPosition
powercode Nov 19, 2020
2c74524
Removing whitespace before/after nullable pragma
powercode Nov 20, 2020
a2e8320
Removing whitespace before/after nullable pragma
powercode Nov 20, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions src/System.Management.Automation/engine/parser/Position.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,12 +15,13 @@ namespace System.Management.Automation.Language
/// <summary>
/// Represents a single point in a script. The script may come from a file or interactive input.
/// </summary>
#nullable enable
public interface IScriptPosition
{
/// <summary>
/// The name of the file, or if the script did not come from a file, then null.
/// </summary>
string File { get; }
string? File { get; }

/// <summary>
/// The line number of the position, with the value 1 being the first line.
Expand All @@ -45,8 +46,9 @@ public interface IScriptPosition
/// <summary>
/// The complete script that this position is included in.
/// </summary>
string GetFullScript();
string? GetFullScript();
}
#nullable restore

/// <summary>
/// Represents the a span of text in a script.
Expand Down