Skip to content

Copier update (fix script again)#27

Merged
ejfine merged 1 commit intomainfrom
rest
May 9, 2025
Merged

Copier update (fix script again)#27
ejfine merged 1 commit intomainfrom
rest

Conversation

@ejfine
Copy link
Contributor

@ejfine ejfine commented May 9, 2025

pull in changes

@ejfine ejfine requested a review from Copilot May 9, 2025 12:48
@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 primarily updates the workflow files to leverage the GitHub REST API for triggering workflow dispatches and bumps the copier template version.

  • Update workflow files to use github.rest.actions.createWorkflowDispatch instead of github.actions.createWorkflowDispatch.
  • Update the _commit version in the copier configuration to align with the latest template changes.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
template/.github/workflows/dependabot-post-update.yaml Updated API call to use GitHub REST client for workflow dispatch.
.github/workflows/dependabot-post-update.yaml Same API update as in template, ensuring consistency across workflows.
.copier-answers.yml Bumped the commit version to reflect updated copier template changes.
Comments suppressed due to low confidence (3)

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

  • Using the GitHub REST client for the workflow dispatch call is appropriate. Please ensure that tests exist to validate the behavior with the new API endpoint.
await github.rest.actions.createWorkflowDispatch({

.github/workflows/dependabot-post-update.yaml:54

  • The change to github.rest.actions.createWorkflowDispatch is correct per updated API practices. Confirm that the change is compatible with all dependent workflows.
await github.rest.actions.createWorkflowDispatch({

.copier-answers.yml:2

  • The updated commit version should be verified against the expected state of the template version. It would be beneficial to document the reason for this revision in the project changelog if not already present.
_commit: v0.0.41-1-gf72b639

@ejfine ejfine merged commit fee151a into main May 9, 2025
8 checks passed
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