Format

Author: hectorhdzg

.github/workflows/pr-review.yml

@@ -6,8 +6,10 @@ on:
     types:
       - submitted
 
+permissions: read-all
+
 jobs:
-  approved:
+  add-owner-approved-label:
     if: github.event.review.state == 'approved'
     runs-on: ubuntu-latest
     steps:
@@ -21,6 +23,6 @@ jobs:
         run: ./.github/workflows/scripts/add-labels-to-reviewed-pr.sh
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-          ISSUE: ${{ github.event.issue.number }}
-          COMMENT: ${{ github.event.comment.body }}
-          SENDER: ${{ github.event.sender.login }}
+          REPO: ${{ github.repository }}
+          PR: ${{ github.event.number }}
+          

.github/workflows/scripts/add-labels-to-reviewed-pr.sh

@@ -1,7 +1,18 @@
 #!/usr/bin/env bash
 #
 # Copyright The OpenTelemetry Authors
-# SPDX-License-Identifier: Apache-2.0
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#      https://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
 #
 # Adds a "has:owner-approval" label to a PR if a reviewer who approved it is a component owner.
 
@@ -17,7 +28,6 @@ main () {
 
     # The latestReviews key returns the latest review for each reviewer cutting out any other reviews.
     JSON=$(gh pr view "${PR}" --json "files,author,latestReviews" | tr -dc '[:print:]' | sed -E 's/\\[a-z]//g')
-    AUTHOR=$(echo -n "${JSON}"| jq -r '.author.login')
     FILES=$(echo -n "${JSON}"| jq -r '.files[].path')
     LATEST_REVIEWS=$(echo -n "${JSON}" | jq -c '.latestReviews')
 
@@ -40,7 +50,6 @@ main () {
             if [[ -v PROCESSED_COMPONENTS["${COMPONENT}"] ]]; then
                 continue
             fi
-
             PROCESSED_COMPONENTS["${COMPONENT}"]=true
 
             # Check if updated file is owned by one of the reviewers"
@@ -63,4 +72,4 @@ main () {
 }
 
 # Ensure the script does not block a PR even if it fails
-main || echo "Failed to run $0"
+main || echo "Failed to run $0"

.github/workflows/scripts/get-codeowners.sh

@@ -1,7 +1,18 @@
 #!/usr/bin/env bash
 #
 # Copyright The OpenTelemetry Authors
-# SPDX-License-Identifier: Apache-2.0
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#      https://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
 #
 # Gets the owners for a given component from the component_owners.yml file.
 

.github/workflows/scripts/get-components.sh

@@ -1,7 +1,18 @@
 #!/usr/bin/env bash
 #
 # Copyright The OpenTelemetry Authors
-# SPDX-License-Identifier: Apache-2.0
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#      https://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
 #
 # Gets the components from the component_owners.yml file.
 
@@ -17,4 +28,4 @@ while IFS= read -r line; do
         COMPONENT=$(echo "$line" | sed -E 's/^[[:space:]]*([^:]+):.*/\1/')
         echo "$COMPONENT"
     fi
-done < "$YML_FILE"
+done < "$YML_FILE"