Skip to content

Conversation

@FidelusAleksander
Copy link
Contributor

@FidelusAleksander FidelusAleksander commented Jun 25, 2025

Changes

This could be useful in some occasions, one that I've run into is people completing the Getting Started with Copilot exercise and only after finishing they notice the bonus step.
The bonus step workflow was still enabled, but it would fail because the find-exercise workflow was only looking for open issues.

https://docs.github.com/en/rest/issues/issues?apiVersion=2022-11-28#list-repository-issues
image

Checklist

  • I have added or updated appropriate labels to this PR
  • I have tested my changes
  • I have updated the documentation if needed

Copilot AI review requested due to automatic review settings June 25, 2025 08:45
@github-actions github-actions bot added the workflows Changes to reusable workflows label Jun 25, 2025
Copy link
Contributor

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 updates the find-exercise workflow so that it no longer filters only open issues, helping to support bonus step workflows that may run on closed issues.

  • Removed the hardcoded "state: 'open'" parameter when listing issues
  • The workflow will now include both open and closed issues in its execution
Comments suppressed due to low confidence (1)

.github/workflows/find-exercise-issue.yml:40

  • Removing the state filter now allows both open and closed issues to be returned. Please ensure that any subsequent logic in the workflow properly handles issues regardless of their state.
              repo: context.repo.repo,

@FidelusAleksander FidelusAleksander changed the title fix: update find-exercise workflow to work on closed issues fix: update find-exercise workflow to also work on closed issues Jun 25, 2025
@FidelusAleksander FidelusAleksander marked this pull request as draft June 25, 2025 09:34
@FidelusAleksander FidelusAleksander marked this pull request as ready for review June 25, 2025 09:37
@FidelusAleksander
Copy link
Contributor Author

FidelusAleksander commented Jun 25, 2025

@chriswblake @arilivigni Do you guys see any problems with this workflow also returning the exercise issue after it is closed?

@FidelusAleksander FidelusAleksander merged commit ebf43bb into main Jun 25, 2025
6 checks passed
@FidelusAleksander FidelusAleksander deleted the FidelusAleksander-patch-1 branch June 25, 2025 10:43
@chriswblake
Copy link
Member

No issues from me. I think that is fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

workflows Changes to reusable workflows

Development

Successfully merging this pull request may close these issues.

4 participants