Skip to content

Commit 160f5fe

Browse files
authored
Small alignments of workflows across the 6 Java repos (#342)
* Align workflows with other OpenTelemetry Java repositories - Add OSSF scorecard comment to gradle-wrapper-validation * Align prepare-release-branch comment to 'for git push to PR branch' Standardize permissions comment for consistency across repositories. * Switch from archived client9/misspell to golangci/misspell * Add concurrency guard to build workflow * Remove concurrency guard from push-only workflow * Align lychee config: use double quotes * Align CODEOWNERS header with standard format * Remove redundant CODEOWNERS format section from header
1 parent 63f093f commit 160f5fe

File tree

5 files changed

+10
-9
lines changed

5 files changed

+10
-9
lines changed

.github/CODEOWNERS

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
1-
# Code owners file.
2-
# This file controls who is tagged for review for any given pull request.
1+
#
2+
# Learn about membership in OpenTelemetry community:
3+
# https://github.com/open-telemetry/community/blob/main/guides/contributor/membership.md
4+
#
35

4-
# For anything not explicitly taken by someone else:
56
* @open-telemetry/java-approvers @open-telemetry/java-instrumentation-approvers

.github/config/lychee.toml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,9 @@ include_fragments = true
1111

1212
remap = [
1313
# workaround for https://github.com/lycheeverse/lychee/issues/1729
14-
'https://github.com/(.*?)/(.*?)/blob/(.*?)/(.*#.*)$ https://raw.githubusercontent.com/$1/$2/$3/$4',
14+
"https://github.com/(.*?)/(.*?)/blob/(.*?)/(.*#.*)$ https://raw.githubusercontent.com/$1/$2/$3/$4",
1515
]
1616

1717
exclude = [
18-
'^https://github.com/.*#discussion_r.*$',
18+
"^https://github.com/.*#discussion_r.*$",
1919
]

.github/workflows/gradle-wrapper-validation.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,4 +13,5 @@ jobs:
1313
steps:
1414
- uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
1515

16+
# this needs to be in its own workflow in order to make OSSF scorecard happy
1617
- uses: gradle/actions/wrapper-validation@4d9f0ba0025fe599b4ebab900eb7f3a1d93ef4c2 # v5.0.0

.github/workflows/prepare-release-branch.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ jobs:
2525
2626
create-pull-request-against-release-branch:
2727
permissions:
28-
contents: write # for Git to git push
28+
contents: write # for git push to PR branch
2929
runs-on: ubuntu-latest
3030
needs:
3131
- prereqs
@@ -86,7 +86,7 @@ jobs:
8686
8787
create-pull-request-against-main:
8888
permissions:
89-
contents: write # for Git to git push
89+
contents: write # for git push to PR branch
9090
runs-on: ubuntu-latest
9191
needs:
9292
- prereqs

.github/workflows/reusable-misspell-check.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,7 @@ jobs:
1414

1515
- name: Install misspell
1616
run: |
17-
curl -L -o install-misspell.sh https://raw.githubusercontent.com/client9/misspell/master/install-misspell.sh
18-
sh ./install-misspell.sh
17+
curl -sfL https://raw.githubusercontent.com/golangci/misspell/master/install-misspell.sh | sh -s -- -b bin
1918
2019
- name: Run misspell
2120
run: bin/misspell -error .

0 commit comments

Comments
 (0)