Граф коммитов

50 Коммитов

Автор SHA1 Сообщение Дата
Coby Allred 809f458c4a
Update Component Detection to .NET v8 (#1249)
* Initial NET 8 changes

* Resolve formatting errors, fix tests

* Fix debug log condition

* Rollback SDK version

* Update csproj refs

* Fix integration test build failures

* Potential test fix

* Set up .NET 6

* Update test csproj
2024-09-27 13:41:06 -07:00
cobya 7528c8f4fd Revert non-PR .NET 8 commits 2024-09-16 09:27:20 -07:00
cobya 7bb2afae00 Initial NET 8 changes 2024-09-10 12:57:23 -07:00
Jamie Magee e55d02302b
Add types for `package-lock.json` versions (#482) 2023-03-16 10:01:49 -07:00
Justin Perez 273d48eb61
feat: Switch to Serilog (#441) 2023-02-23 12:28:08 -08:00
Jamie Magee f6912c0258
refactor: migrate away from MEF to Dependency Injection (#412) 2023-02-14 09:33:07 -08:00
Justin Perez 4e49e43c89
Fix some analyzer warnings (#416) 2023-01-31 08:43:20 -08:00
Rushabh 2a128f69f0
fix analyzer warnings: SA1210, SA1414, VSTHRD002, CA1849, VSTHRD103, VSTHRD104 (#341) 2022-11-02 11:51:49 -07:00
Daniel Akili 3da9140347
fix: CA1829, CA1834, CA1846, CA1847 (#338)
* CA1829

* CA1834

* CA1846, CA1847

* updated if statement condition

* added JSON002 as suggestion

* pr snapshot

* spdx elements count

* removed JSON22 suggestion

* if statement syntax update

Co-authored-by: Jamie Magee <jamagee@microsoft.com>

* updated if statement

* if statement

* removed extra '?'

* verifying snapshots early success

* short if statement syntax

Co-authored-by: Daniel Akili <danielakili@microsoft.com>
Co-authored-by: Jamie Magee <jamagee@microsoft.com>
2022-11-01 11:46:00 -07:00
Daniel Akili 92cc2ab3bc
fix: CA1308 (#335)
* CA1002

* CA1308

* pr build check

* _

* pr build check2

* CA1308

* fixed conversation suggestions

Co-authored-by: Daniel Akili <danielakili@microsoft.com>
2022-10-28 15:28:02 -07:00
Rushabh 59a5d5732b
Fix Analyzer warnings: CA1707, CA1300, CA1040 (#337)
* fix CA1707: Identifiers should not contain underscores

* fix  CA1040: Avoid empty interfaces

* fixed SA1300: Element Must Begin With Upper Case Letter
2022-10-28 15:09:18 -07:00
Daniel Akili 2e911a558e
fix: analyzer warnings IDE0019, IDE0021, IDE0037 (#326) 2022-10-27 17:04:50 -07:00
Coby Allred 242d165db0
Add IDE0052 to warning exclusion (#329)
Add IDE0052 to warning exclusion
2022-10-25 10:58:26 -07:00
Coby Allred bf56f35ac4
Add new exclusions to .editorconfig (#328)
Add the IDE0053, IDE0200, CA1311, CA1852, and CA1854 exlcusions to .editorconfig to avoid build breaks.
2022-10-25 10:10:08 -07:00
Rushabh 62133a4f36
refactor: fix analyzer warnings (#319)
* fix IDE0120: Simplify LINQ expression

* fixed IDE0037: Member name can be simplified

* fix: IDE0071: Interpolation can be simplified

* fixed IDE0052: Private member can be removed as the value assigned to it is never read

* fixed IDE0032: Use auto property

* IDE0054: Use compound assignment

* fixed SA1203: connstant fields should not be after the non-constant

* fixed IDE0037 by reformatting

* fixed SA1316: Tuple element names should use correct casing

* fixed SA1216:  Using static directives should be placed at the correct location

* fixed SA1401: field should be private
2022-10-25 09:12:07 -07:00
Rushabh e90745d29f
added verification for dockerfile and spdx detector, minor bug fixes (#322) 2022-10-24 09:13:34 -07:00
Rushabh 8e943778bd
fix analyzer warnings: IDE0024, IDE0039, IDE0066, IDE0010, IDE0074, IDE0075, SA1500, SA1515, CA1827, IDE0044 (#317)
* fix IDE0024: Use expression body for operators

* fix IDE0039: Use local function

* fix IDE0066: Use 'switch' expression

* fix IDE0010: Populate switch

* fix IDE0074: Use compound assignment

* fix  IDE0075: Conditional expression can be simplified

* fix SA1500: Braces for multi-line statements should not share line

* fix SA1515: Single-line comment should be preceded by blank line

* fix CA1827: Do not use Count() or LongCount() when Any() can be used

* fix  IDE0044: Add readonly modifier
2022-10-20 14:11:42 -07:00
Rushabh b1532df7a8
Fix analyzer warnings: SA1142, SA1201 (#308) 2022-10-19 09:35:56 -07:00
Rushabh bd5c65b62a
fix IDE0031, IDE0042, IDE0018, IDE0017, IDE0004, IDE0007, IDE0016 (#305) 2022-10-18 13:33:00 -07:00
Rushabh 14a9a60aae
refactor: fix IDE0009, IDE0033, IDE0150, IDE0051, SA1028 (#303) 2022-10-17 15:29:03 -07:00
Jamie Magee b2439a3c67
refactor: fix CA1820 (#302) 2022-10-17 14:09:26 -07:00
Rushabh c2086cbdb4
Fix release failure 2022-10-07 09:42:07 -07:00
Rushabh f246185d70
Migration component detection to .net6 (#288)
Co-authored-by: Sebstien Lebreton <sebastien@lebreton.fr>
Co-authored-by: Jamie Magee <jamagee@microsoft.com>
Co-authored-by: Jamie Magee <jamie.magee@gmail.com>
2022-10-06 15:38:05 -07:00
Adam Plaskitt bef8d6149e
Refactor: Fix warning SA1101 (#236)
Co-authored-by: amitla1 <46578839+amitla1@users.noreply.github.com>
Co-authored-by: Jamie Magee <jamie.magee@gmail.com>
2022-10-05 17:27:24 +00:00
Omotola eeae9dad67
Analyzer Warnings SA-1614, SA-1615, SA-1616, SA-1617 (#262)
Added param, and return value documentations to fix Analyzer warnings SA1614, SA1615, SA1616, SA1617
2022-09-09 15:09:52 -07:00
amitla1 7d5668e816
Refactor: Fix SA1202 (#252)
* Fis SA1202

* removed whitespac

* reverted this file

* reverted this file

* Reverted this file

* Fixing indentation

* Removed whitespace

* Removed whitespace

* changed location of static variable

* took out the SA1202 suggestion

* changed private method to come after public methods

* Added supression message for access level order

Co-authored-by: Amitla Vannikumar <avannikumar@microsoft.com>
2022-09-06 12:59:11 -07:00
amitla1 21bf695df1
Refactor: Fix SA1214 (#250)
Co-authored-by: Amitla Vannikumar <avannikumar@microsoft.com>
2022-09-02 10:26:34 -07:00
Adam Plaskitt 799c30a3ae
Refactor: Fix warning SA1108 (#237)
* Related to #202

Block statements should not contain embedded comments
https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/documentation/SA1108.md

* Remove trailing whitespace warning

Co-authored-by: amitla1 <46578839+amitla1@users.noreply.github.com>
2022-09-01 13:15:39 -07:00
amitla1 165275ed0a
Refactor: Fix SA1413 (#249)
* Fix SA1413

* Update BcdeScanExecutionServiceTests.cs

* Fixed merge conflicts

* Update YarnParserTests.cs

* Update BcdeScanExecutionServiceTests.cs

* Update YarnParserTests.cs

Co-authored-by: Amitla Vannikumar <avannikumar@microsoft.com>
2022-08-31 17:07:35 -07:00
amitla1 e19514011c
Fix SA1515 (#248)
Co-authored-by: Amitla Vannikumar <avannikumar@microsoft.com>
2022-08-31 15:45:28 -07:00
Adam Plaskitt 00bca26514
Remove fix occurances warning SA1114 (#238)
Parameter list should follow declaration.
https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/documentation/SA1114.md

Related to #202
2022-08-30 09:48:07 -07:00
Jamie Magee 83acbd4788
refactor: fix SA1210 (#227) 2022-08-25 09:14:39 -07:00
Jamie Magee d1c6d005c0
refactor: fix SA1629 (#226)
Related to #202

Documentation text should end with a period
https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/documentation/SA1629.md
2022-08-25 09:13:34 -07:00
Jamie Magee f71cc4146b
refactor: fix SA1135 (#225) 2022-08-25 09:13:20 -07:00
Jamie Magee bbca187645
refactor: fix SA1500 (#223)
Related to #202

Braces for multi-line statements should not share line
https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/documentation/SA1500.md
2022-08-24 16:23:34 -07:00
Jamie Magee 644512a17f
refactor: fix SA1208 (#218) 2022-08-24 11:42:33 -07:00
amitla1 c71e1ea5da
Refactor: Fix SA1204 (#219)
Co-authored-by: Amitla Vannikumar <avannikumar@microsoft.com>
2022-08-24 11:36:10 -07:00
amitla1 8e8c3a4557
Refactor: Fix SA1117 (#214)
Co-authored-by: Amitla Vannikumar <avannikumar@microsoft.com>
2022-08-19 16:56:29 -07:00
amitla1 9458dc569e
Refactor: Fix SA1012, SA1013, SA1026 and SA1118 (#213)
Co-authored-by: Jamie Magee <jamie.magee@gmail.com>
Co-authored-by: Amitla Vannikumar <avannikumar@microsoft.com>
2022-08-19 22:18:02 +00:00
amitla1 027bc286e6
Refactor: Fix SA1008 (#212)
Co-authored-by: Amitla Vannikumar <avannikumar@microsoft.com>
2022-08-19 12:44:12 -07:00
amitla1 02f9095914
Refactor: fix SA1011 (#211)
Co-authored-by: Amitla Vannikumar <avannikumar@microsoft.com>
2022-08-19 19:03:41 +00:00
amitla1 57b573a07a
Refactor: fix SA1009 (#210)
Co-authored-by: Amitla Vannikumar <avannikumar@microsoft.com>
2022-08-19 11:08:39 -07:00
Jamie Magee 3571344f10
refactor: fix SA1105 (#206) 2022-08-19 09:54:47 -07:00
Jamie Magee aa4d079294
refactor: fix SA1137 (#209) 2022-08-19 09:49:50 -07:00
Jamie Magee 368f3c9692
refactor: fix SA1518 (#208) 2022-08-17 16:08:33 +00:00
Jamie Magee a66a444f3e
refactor: fix SA1648 (#207) 2022-08-17 09:04:51 -07:00
Jamie Magee a99550f765
refactor: fix SA1005 (#205)
Related to #202

Single line comment should begin with a space
https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/documentation/SA1005.md
2022-08-16 12:13:48 -07:00
Jamie Magee 981c4ed2bc
refactor: fix SA1001 (#204) 2022-08-16 11:52:44 -07:00
Jamie Magee 06e5188844
refactor: enable `TreatWarningsAsErrors` (#203) 2022-08-16 11:46:52 -07:00
Jamie Magee 0319816fab
chore: codestyle and roslyn analyzers best practices (#171) 2022-07-15 11:58:05 -07:00