Skip to content

Copier update (trigger)#25

Merged
ejfine merged 3 commits intomainfrom
trigger
May 9, 2025
Merged

Copier update (trigger)#25
ejfine merged 3 commits intomainfrom
trigger

Conversation

@ejfine
Copy link
Contributor

@ejfine ejfine commented May 9, 2025

Pull in fix

@ejfine ejfine requested a review from Copilot May 9, 2025 12:28
@ejfine ejfine self-assigned this May 9, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes the Dependabot workflow by updating permissions and adding a manual CI dispatch step using GitHub Script, while also bumping dependency versions and updating the Copier configuration.

  • Added "actions: write" permission and a CI dispatch step to workaround GITHUB_TOKEN limitations.
  • Updated dependency versions in workflow and context files.
  • Updated the Copier commit version.

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
template/.github/workflows/dependabot-post-update.yaml Added extra permission and a new step to dispatch CI via GitHub Script.
extensions/context.py Updated the context with the new GitHub Script version.
.github/workflows/dependabot-post-update.yaml Implemented similar workflow changes as in the template version.
.copier-answers.yml Updated the Copier configuration commit version.
Comments suppressed due to low confidence (1)

template/.github/workflows/dependabot-post-update.yaml:56

  • [nitpick] Confirm that specifying 'ci.yaml' as the workflow ID targets the intended CI workflow, particularly if workflow naming might change in the future.
workflow_id: "ci.yaml",

uses: actions/github-script@v7.0.1
with:
github-token: ${{ secrets.GITHUB_TOKEN }}
script: |
Copy link

Copilot AI May 9, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ensure 'context.payload.workflow_run' is reliably defined before accessing its properties to prevent potential runtime exceptions; consider adding a guard clause if this scenario can occur.

Suggested change
script: |
script: |
if (!context.payload.workflow_run) {
throw new Error("workflow_run is not defined in the event payload; aborting dispatch.");
}

Copilot uses AI. Check for mistakes.
@ejfine ejfine merged commit 2c060a5 into main May 9, 2025
9 checks passed
@ejfine ejfine deleted the trigger branch May 9, 2025 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants