Skip to content

Commit

Permalink
Merge pull request #1168 from godaddy/pre-commit-cleanup
Browse files Browse the repository at this point in the history
exclude generated source from pre-commit hooks, global cleanup via `pre-commit run --all`
  • Loading branch information
aka-bo authored Jan 13, 2025
2 parents 8e6c8a0 + 884ca22 commit 05731a9
Show file tree
Hide file tree
Showing 7 changed files with 4 additions and 6 deletions.
1 change: 0 additions & 1 deletion .github/workflows/go-securememory-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -101,4 +101,3 @@ jobs:
with:
paths: 'go/securememory/junit_results.xml'
if: always()

1 change: 0 additions & 1 deletion .github/workflows/java-securememory-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -90,4 +90,3 @@ jobs:
with:
paths: 'java/secure-memory/target/surefire-reports/TEST-*.xml'
if: always()

1 change: 1 addition & 0 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
exclude: '^.*generated.*$'
repos:
- repo: https://github.com/pre-commit/pre-commit-hooks
rev: v5.0.0
Expand Down
2 changes: 1 addition & 1 deletion csharp/AppEncryption/AppEncryption/AppEncryption.csproj
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<Project Sdk="Microsoft.NET.Sdk">
<Project Sdk="Microsoft.NET.Sdk">
<PropertyGroup>
<PackageId>GoDaddy.Asherah.AppEncryption</PackageId>
<Title>AppEncryption</Title>
Expand Down
2 changes: 1 addition & 1 deletion csharp/SecureMemory/SecureMemory/SecureMemory.csproj
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<Project Sdk="Microsoft.NET.Sdk">
<Project Sdk="Microsoft.NET.Sdk">

<PropertyGroup>
<PackageId>GoDaddy.Asherah.SecureMemory</PackageId>
Expand Down
2 changes: 1 addition & 1 deletion java/app-encryption/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,7 @@ AwsKmsClientFactory awsKmsClientFactory = ...;
KeyManagementService keyManagementService = AwsKeyManagementServiceImpl.newBuilder(regionMap, "us-east-1")
.withAwsKmsClientFactory(awsKmsClientFactory)
.build();
```
```

#### Static KMS (FOR TESTING ONLY)

Expand Down
1 change: 0 additions & 1 deletion samples/csharp/ReferenceApp/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -175,4 +175,3 @@ follows:

--version Display version information.
```

0 comments on commit 05731a9

Please sign in to comment.