Merge branch 'develop' into feat/parameter-env-variable

Author: leandrodamascena

.github/scripts/constants.js

@@ -24,6 +24,9 @@ module.exports = Object.freeze({
     /** @type {string} */
     "LABEL_BLOCK_REASON": "need-issue",
 
+    /** @type {string} */
+    "LABEL_BLOCK_MISSING_LICENSE_AGREEMENT": "need-license-agreement-acknowledge",
+
     /** @type {string} */
     "LABEL_PENDING_RELEASE": "pending-release",
 

.github/scripts/label_missing_acknowledgement_section.js

@@ -0,0 +1,41 @@
+const {
+  PR_ACTION,
+  PR_AUTHOR,
+  PR_BODY,
+  PR_NUMBER,
+  IGNORE_AUTHORS,
+  LABEL_BLOCK,
+  LABEL_BLOCK_MISSING_LICENSE_AGREEMENT
+} = require("./constants")
+
+module.exports = async ({github, context, core}) => {
+    if (IGNORE_AUTHORS.includes(PR_AUTHOR)) {
+      return core.notice("Author in IGNORE_AUTHORS list; skipping...")
+    }
+
+    if (PR_ACTION != "opened") {
+      return core.notice("Only newly open PRs are labelled to avoid spam; skipping")
+    }
+
+    const RELATED_ACK_SECTION_REGEX = /By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice./;
+
+    const isMatch = RELATED_ACK_SECTION_REGEX.exec(PR_BODY);
+    if (isMatch == null) {
+        core.info(`No acknowledgement section found, maybe the author didn't use the template but there is one.`)
+
+        let msg = "No acknowledgement section found. Please make sure you used the template to open a PR and didn't remove the acknowledgment section. Check the template here: https://github.com/awslabs/aws-lambda-powertools-python/blob/develop/.github/PULL_REQUEST_TEMPLATE.md#acknowledgment";
+        await github.rest.issues.createComment({
+          owner: context.repo.owner,
+          repo: context.repo.repo,
+          body: msg,
+          issue_number: PR_NUMBER,
+        });
+
+        return await github.rest.issues.addLabels({
+          issue_number: PR_NUMBER,
+          owner: context.repo.owner,
+          repo: context.repo.repo,
+          labels: [LABEL_BLOCK, LABEL_BLOCK_MISSING_LICENSE_AGREEMENT]
+        })
+    }
+}

.github/workflows/on_opened_pr.yml

@@ -32,3 +32,20 @@ jobs:
           script: |
             const script = require('.github/scripts/label_missing_related_issue.js')
             await script({github, context, core})
+  check_acknowledge_section:
+    needs: get_pr_details
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+      - name: "Ensure acknowledgement section is present"
+        uses: actions/github-script@v6
+        env:
+          PR_BODY: ${{ needs.get_pr_details.outputs.prBody }}
+          PR_NUMBER: ${{ needs.get_pr_details.outputs.prNumber }}
+          PR_ACTION: ${{ needs.get_pr_details.outputs.prAction }}
+          PR_AUTHOR: ${{ needs.get_pr_details.outputs.prAuthor }}
+        with:
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+          script: |
+            const script = require('.github/scripts/label_missing_acknowledgement_section.js')
+            await script({github, context, core})

.github/workflows/python_build.yml

@@ -53,7 +53,7 @@ jobs:
       - name: Complexity baseline
         run: make complexity-baseline
       - name: Upload coverage to Codecov
-        uses: codecov/codecov-action@d9f34f8cd5cb3b3eb79b3e4b5dae3a16df499a70 # 3.1.1
+        uses: codecov/codecov-action@40a12dcee2df644d47232dde008099a3e9e4f865 # 3.1.2
         with:
           file: ./coverage.xml
           env_vars: PYTHON

.github/workflows/reusable_update_v2_layer_arn_docs.yml

@@ -46,7 +46,7 @@ jobs:
         run: |
           HAS_CHANGE=$(git status --porcelain)
           test -z "${HAS_CHANGE}" && echo "Nothing to update" && exit 0
-          git add docs/index.md
+          git add docs/index.md docs/core/logger.md docs/core/tracer.md docs/core/metrics.md
           git commit -m "chore: update v2 layer ARN on documentation"
           git pull origin "${BRANCH}" # prevents concurrent branch update failing push
           git push origin HEAD:refs/heads/"${BRANCH}"