Skip to content

Commit 59a349f

Browse files
Merge pull request #89 from Taritsyn/fix-warnings
Fix a warnings [release]
2 parents 485999a + 2930ab0 commit 59a349f

File tree

1 file changed

+9
-24
lines changed

1 file changed

+9
-24
lines changed

src/WebOptimizerScssOptions.cs

Lines changed: 9 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -10,23 +10,16 @@ public class WebOptimizerScssOptions
1010
/// <summary>
1111
/// Gets or sets the maximum number of digits after the decimal. Default is 5.
1212
/// </summary>
13-
/// <remarks>
14-
/// TODO: the C function is not working
15-
/// </remarks>
1613
public int Precision { get; set; } = 5;
1714

1815
/// <summary>
19-
/// Gets or sets the output style. Default is <see cref="ScssOutputStyle.Nested"/>
16+
/// Gets or sets the output style. Default is <see cref="OutputStyle.Expanded"/>
2017
/// </summary>
2118
public OutputStyle OutputStyle { get; set; } = OutputStyle.Expanded;
2219

2320
/// <summary>
24-
/// Gets or sets a value indicating whether to generate source map (result in <see cref="ScssResult.SourceMap"/>
21+
/// Gets or sets a value indicating whether to generate source map. Default is <c>false</c>
2522
/// </summary>
26-
/// <remarks>
27-
/// Note that <see cref="OutputFile"/> should be setup. <see cref="SourceMapFile"/> will then automatically
28-
/// map to <see cref="OutputFile"/> + ".map" unless specified.
29-
/// </remarks>
3023
public bool GenerateSourceMap { get; set; } = false;
3124

3225
/// <summary>
@@ -47,18 +40,15 @@ public class WebOptimizerScssOptions
4740
/// <summary>
4841
/// Gets or sets a value indicating whether to enable or disable the inclusion of source map information in the output file. Default is <c>false</c>
4942
/// </summary>
50-
/// <remarks>
51-
/// If this is set to <c>true</c>, the <see cref="OutputFile"/> must be setup to avoid unexpected behavior.
52-
/// </remarks>
5343
public bool OmitSourceMapUrl { get; set; } = false;
5444

5545
/// <summary>
56-
/// Gets or sets a value indicating whether the scss content to transform is using indented syntax.
46+
/// Gets or sets a value indicating whether the scss content to transform is using indented syntax. Default is <c>false</c>
5747
/// </summary>
58-
public bool IsIndentedSyntaxSource { get; set; }
48+
public bool IsIndentedSyntaxSource { get; set; } = false;
5949

6050
/// <summary>
61-
/// Gets or sets the indent string. Default is 2 spaces.
51+
/// Gets or sets the indent string. Default is 2 spaces
6252
/// </summary>
6353
public string Indent { get; set; } = " ";
6454

@@ -68,22 +58,17 @@ public class WebOptimizerScssOptions
6858
public string Linefeed { get; set; } = "\n";
6959

7060
/// <summary>
71-
/// Gets or sets the value that will be emitted as sourceRoot in the source map information. Default is null.
61+
/// Gets or sets the value that will be emitted as <c>sourceRoot</c> in the source map information. Default is empty string
7262
/// </summary>
73-
public string SourceMapRoot { get; set; }
63+
public string SourceMapRoot { get; set; } = string.Empty;
7464

7565
/// <summary>
76-
/// Gets the include paths that will be used to search for @import directives in scss content.
66+
/// Gets the include paths that will be used to search for @import directives in scss content. Default is empty list
7767
/// </summary>
7868
public List<string> IncludePaths { get; } = [];
7969

80-
///// <summary>
81-
///// Gets or sets a dynamic delegate used to resolve imports dynamically.
82-
///// </summary>
83-
//public TryImportDelegate TryImport { get; set; }
84-
8570
/// <summary>
86-
/// Gets or sets the option to minify the resulting css.
71+
/// Gets or sets the option to minify the resulting css. Default is <c>true</c>
8772
/// </summary>
8873
public bool MinifyCss { get; set; } = true;
8974
}

0 commit comments

Comments
 (0)