Skip to content

Separate issue management from project management - #236

Merged
chicco785 merged 3 commits into
mainfrom
test-workflow-call
Jan 26, 2026
Merged

Separate issue management from project management#236
chicco785 merged 3 commits into
mainfrom
test-workflow-call

Conversation

@chicco785

@chicco785 chicco785 commented Jan 26, 2026

Copy link
Copy Markdown
Member

Description

This PR, to solve scheduling issues, separates issue management jobs from project management jobs.

Changes Made

  • Add a new usable workflow dedicated to issue management.

Related Issues

N/A

Checklist

  • I have used a PR title that is descriptive enough for a release note.
  • I have tested these changes locally.
  • I have added appropriate tests or updated existing tests.
  • I have tested these changes on a cluster [name of the cluster] / customer [name of the customer]
  • I have added appropriate documentation or updated existing documentation.

@chicco785
chicco785 requested a review from Copilot January 26, 2026 07:21

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 a workflow trigger condition issue by restricting two jobs to only run when called by another workflow. Previously, these jobs could be triggered by schedule, manual dispatch, or workflow call events, but they are now limited to only workflow call events.

Changes:

  • Updated trigger conditions for manage-stale-issues and sync-priority jobs to only respond to workflow_call events

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@chicco785

Copy link
Copy Markdown
Member Author

@copilot why those workflows are not triggered from this caller using schedule, but works with workflow_dispatch?

name: Project Management
on:
issues:
types:
- labeled
pull_request:
branches: ["main"]
types:
- labeled
- ready_for_review
schedule:
- cron: "*/5 * * * *"
workflow_dispatch:
jobs:
add-to-project:
uses: zaphiro-technologies/github-workflows/.github/workflows/add-to-project.yaml@test-workflow-call
secrets: inherit

Copilot AI commented Jan 26, 2026

Copy link
Copy Markdown
Contributor

@chicco785 I've opened a new pull request, #237, to work on those changes. Once the pull request is ready, I'll request review from you.

@chicco785 chicco785 changed the title Fix workflow not triggered by caller Separate issue management from project management Jan 26, 2026
@chicco785

Copy link
Copy Markdown
Member Author

@copilot update the documentation and verify my changes

Copilot AI commented Jan 26, 2026

Copy link
Copy Markdown
Contributor

@chicco785 I've opened a new pull request, #238, to work on those changes. Once the pull request is ready, I'll request review from you.

Updated README.MD to reflect the separation of issue management from
project management workflows introduced in commit 99c09e6.

## Changes

- **Documented `add-to-project.yaml`**: Handles project operations
(adding issues/PRs to projects, assigning reviewers). Triggers:
`issues`, `pull_request`, `workflow_call`

- **Documented `manage-issues.yaml`**: New workflow for issue management
with two jobs:
- `manage-stale-issues`: Marks unplanned issues as stale, closes after
grace period
  - `sync-priority`: Maps GitHub Project priority fields to P0-P3 labels
  
  Triggers: `schedule` (daily), `workflow_dispatch`, `workflow_call`

The separation resolves scheduling conflicts - issue management jobs now
execute on schedule without restrictions from event-triggered project
management operations.

<!-- START COPILOT CODING AGENT TIPS -->
---

✨ Let Copilot coding agent [set things up for
you](https://github.com/zaphiro-technologies/github-workflows/issues/new?title=✨+Set+up+Copilot+instructions&body=Configure%20instructions%20for%20this%20repository%20as%20documented%20in%20%5BBest%20practices%20for%20Copilot%20coding%20agent%20in%20your%20repository%5D%28https://gh.io/copilot-coding-agent-tips%29%2E%0A%0A%3COnboard%20this%20repo%3E&assignees=copilot)
— coding agent works faster and does higher quality work when set up for
your repo.

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: chicco785 <777218+chicco785@users.noreply.github.com>
@chicco785 chicco785 added the ci This issue/PR is about continuous integration. label Jan 26, 2026
@chicco785
chicco785 marked this pull request as ready for review January 26, 2026 12:44
@chicco785
chicco785 requested a review from cosimomeli January 26, 2026 12:44
Comment thread .github/workflows/manage-issues.yaml
Comment thread .github/workflows/manage-issues.yaml
Comment thread .github/workflows/manage-issues.yaml
Comment thread .github/workflows/manage-issues.yaml
Comment thread .github/workflows/manage-issues.yaml
Comment thread .github/workflows/manage-issues.yaml
Comment thread .github/workflows/add-to-project.yaml
@chicco785
chicco785 enabled auto-merge (squash) January 26, 2026 12:46
@chicco785
chicco785 merged commit 8e5d5c6 into main Jan 26, 2026
23 checks passed
@chicco785
chicco785 deleted the test-workflow-call branch January 26, 2026 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci This issue/PR is about continuous integration.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants