Skip to content

Commit c9f5f2a

Browse files
authored
Add sdk-linting-extensions package (#45949)
Add sdk-linting-extensions package
1 parent a512e10 commit c9f5f2a

File tree

487 files changed

+3487
-8702
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

487 files changed

+3487
-8702
lines changed

.vscode/cspell.json

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -422,6 +422,7 @@
422422
"vmware",
423423
"vnet",
424424
"VNET",
425+
"vnext",
425426
"westcentralus",
426427
"westus",
427428
"WINDOWSVMIMAGE",
@@ -1473,6 +1474,13 @@
14731474
"RESOURCEID"
14741475

14751476
]
1477+
},
1478+
{
1479+
"filename": "sdk/tools/linting-extensions/**",
1480+
"words": [
1481+
"elist",
1482+
"slist"
1483+
]
14761484
}
14771485
],
14781486
"allowCompoundWords": true

CONTRIBUTING.md

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -89,22 +89,6 @@ Some live tests may have additional steps for setting up live testing resources.
8989
See the CONTRIBUTING.md file for the service you wish to test for additional
9090
information or instructions.
9191

92-
### Workaround for Checkstyle error
93-
94-
When building locally you might run into a Checkstyle such as the following:
95-
96-
```
97-
Execution default of goal org.apache.maven.plugins:maven-checkstyle-plugin:3.1.0:check failed:
98-
Plugin org.apache.maven.plugins:maven-checkstyle-plugin:3.1.0 or one of its dependencies could not be resolved:
99-
Could not find artifact com.azure:sdk-build-tools:jar:1.0.0 in ossrh (https://oss.sonatype.org/content/repositories/snapshots/)
100-
```
101-
102-
This is because the `sdk-build-tools` project isn't released to Maven. To resolve this issue you'll need to copy the `eng` folder locally then install `sdk-build-tools`.
103-
104-
`mvn clean install -f eng/code-quality-reports/pom.xml`
105-
106-
All code in the Azure SDKs for Java repository must pass Checkstyle before being merged. The `sdk-build-tools` is updated periodically, so if a new branch fails Checkstyle you'll need to reinstall.
107-
10892
## Versions and versioning
10993

11094
Tooling has been introduced to centralize versioning and help ease the pain of updating artifact versions in POM and README files. Under the eng\versioning directory there exists a version text file for libraries ([version_client.txt](https://github.com/Azure/azure-sdk-for-java/blob/main/eng/versioning/version_client.txt)).

eng/code-quality-reports/README.md

Lines changed: 0 additions & 6 deletions
This file was deleted.

eng/code-quality-reports/ci.yml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,10 +9,12 @@ trigger:
99
paths:
1010
include:
1111
- .vscode/eclipse-format-azure-sdk-for-java.xml
12-
- eng/code-quality-reports/
12+
- eng/code-quality-reports/ci.yml
13+
- eng/lintingconfigs/
1314
- eng/pipelines/code-quality-reports.yml
1415
- eng/pipelines/scripts/Get-Linting-Commands.ps1
1516
- eng/pipelines/scripts/Get-Linting-Reports.ps1
17+
- sdk/tools/linting-extensions/
1618

1719
pr:
1820
branches:
@@ -24,10 +26,12 @@ pr:
2426
paths:
2527
include:
2628
- .vscode/eclipse-format-azure-sdk-for-java.xml
27-
- eng/code-quality-reports/
29+
- eng/code-quality-reports/ci.yml
30+
- eng/lintingconfigs/
2831
- eng/pipelines/code-quality-reports.yml
2932
- eng/pipelines/scripts/Get-Linting-Commands.ps1
3033
- eng/pipelines/scripts/Get-Linting-Reports.ps1
34+
- sdk/tools/linting-extensions/
3135

3236
extends:
3337
template: /eng/pipelines/code-quality-reports.yml

eng/code-quality-reports/src/main/java/com/azure/tools/checkstyle/checks/HttpPipelinePolicyCheck.java

Lines changed: 0 additions & 99 deletions
This file was deleted.

eng/code-quality-reports/src/main/java/com/azure/tools/checkstyle/checks/RawExceptionThrowCheck.java

Lines changed: 0 additions & 74 deletions
This file was deleted.

eng/code-quality-reports/src/main/java/com/azure/tools/checkstyle/checks/ServiceClientBuilderCheck.java

Lines changed: 0 additions & 120 deletions
This file was deleted.

0 commit comments

Comments
 (0)