Merge pull request #128 from Lombiq/issue/OSOE-470

OSOE-470: Remove false entries (actual typos) from spell checking allow lists
Lombiq · Dec 12, 2022 · c6bcfde · c6bcfde
2 parents dc27967 + a4e6100
commit c6bcfde
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 5 deletions.
diff --git a/Lombiq.DataTables/Assets/Scripts/lombiq-datatables.js b/Lombiq.DataTables/Assets/Scripts/lombiq-datatables.js
@@ -21,7 +21,7 @@
             lengthChange: true,
             scrollX: true,
             dom: "<'row dataTables_buttons'<'col-md-12'B>>" +
-                "<'row dataTables_controls'<'col-md-6 dataTables_length'l><'col-md-6 dataTables_search'f>>" +
+                "<'row dataTables_controls'<'col-md-6 dataTables_length'l><'col-md-6 dataTables_search'f>>" + // #spell-check-ignore-line
                 "<'row dataTables_content'<'col-md-12't>>" +
                 "<'row dataTables_footer'<'col-md-12'ip>>",
             buttons: useDefaultButtons,

diff --git a/Lombiq.DataTables/ResourceManagementOptionsConfiguration.cs b/Lombiq.DataTables/ResourceManagementOptionsConfiguration.cs
@@ -90,7 +90,7 @@ static ResourceManagementOptionsConfiguration()
         _manifest
             .DefineScript(ResourceNames.DataTables.AutoInit)
             .SetDependencies(ResourceNames.DataTables.Bootstrap4, ResourceNames.DataTables.Bootstrap4Buttons)
-            .SetUrl(Lombiq + "jquery-datatables-autoinit.js")
+            .SetUrl(Lombiq + "jquery-datatables-autoinit.js") // #spell-check-ignore-line
             .SetVersion("1.0");
 
         _manifest

diff --git a/Lombiq.DataTables/Services/DataTableDataProviderBase.cs b/Lombiq.DataTables/Services/DataTableDataProviderBase.cs
@@ -94,7 +94,8 @@ protected static IEnumerable<DataTableRow> SubstituteByColumn(
                 .ToDictionary(
                     cell => cell.Name,
                     cell => cell.Regex is { } regex
-                        ? new JValue(cell.Token?.ToString().RegexReplace(regex.From, regex.To, RegexOptions.Singleline) ?? string.Empty)
+                        ? new JValue(cell.Token?.ToString().RegexReplace(regex.From, regex.To, RegexOptions.Singleline) // #spell-check-ignore-line
+                            ?? string.Empty)
                         : cell.Token)));
 
     protected async Task RenderLiquidAsync(IEnumerable<DataTableRow> rowList, IList<string> liquidColumns)

diff --git a/Lombiq.DataTables/Tests/Lombiq.DataTables.Tests/UnitTests/Services/LiquidTests.cs b/Lombiq.DataTables/Tests/Lombiq.DataTables.Tests/UnitTests/Services/LiquidTests.cs
@@ -108,7 +108,7 @@ public static IEnumerable<object[]> Data()
             new[]
             {
                 ("Num||^.*$||{{ '$0' | date: '%m/%d/%Y' }}", "Dates", true),
-                ("Cls||^.*$||{{ '$0' | downcase }}", "Magic Words", true),
+                ("Cls||^.*$||{{ '$0' | downcase }}", "Magic Words", true), // #spell-check-ignore-line
             },
             $"{today},foo bar baz;01/01/1970,lorem ipsum dolor sit amet;12/31/2020,the quick brown fox"
                 .Split(';')

diff --git a/Lombiq.DataTables/Views/Lombiq.DataTable.cshtml b/Lombiq.DataTables/Views/Lombiq.DataTable.cshtml
@@ -101,7 +101,7 @@
         $(function () {
             var templates = {};
             $(".@templatesElementName > div[data-id]").each(function(index, element) {
-                templates[element.getAttribute('data-id')] = element.innerHTML.replace(/%7B%7Bdata%7D%7D/g, '@templatePlaceholder');
+                templates[element.getAttribute('data-id')] = element.innerHTML.replace(/%7B%7Bdata%7D%7D/g, '@templatePlaceholder'); // #spell-check-ignore-line
                 element.parentElement.removeChild(element);
             });