Skip to content

fix(CI): Check if linked issue is epic#42

Open
Jkuzz wants to merge 6 commits intomainfrom
fix/linked-epics
Open

fix(CI): Check if linked issue is epic#42
Jkuzz wants to merge 6 commits intomainfrom
fix/linked-epics

Conversation

@Jkuzz
Copy link

@Jkuzz Jkuzz commented Nov 20, 2025

Zenhub no longer performs linking to epics. We can now replace this behaviour by using the pull request connection.

This PR replaces the previous check by checking if there is a linked issue, and if it is an epic, the epic doesn't need an estimate, but the PR does instead. This lets us mark ad-hoc PRs that belong to a project using the issue connections.

image

@Jkuzz Jkuzz requested review from Copilot and mtrunkat November 20, 2025 15:19
@Jkuzz Jkuzz added the adhoc Ad-hoc unplanned task added during the sprint. label Nov 20, 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 updates the CI check for Zenhub epic linking to accommodate changes in Zenhub's API. Instead of checking for linked epics through timeline events, it now checks if a linked issue is itself an epic by querying the issue type directly.

  • Adds a new GraphQL query ZENHUB_ISSUE_TYPE_QUERY to fetch issue types from Zenhub
  • Implements isIssueLinkedToEpic function to determine if a linked issue is an epic
  • Refactors ensureCorrectLinkingAndEstimates to use the new epic checking approach and removes dependency on getLinkedEpics

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

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@github-actions github-actions bot added the t-discovery Issues owned by the discovery team. label Nov 21, 2025
@Jkuzz Jkuzz requested a review from jirimoravcik November 26, 2025 09:05
Copy link
Member

@mtrunkat mtrunkat left a comment

Choose a reason for hiding this comment

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

One naming comment otherwise looks good

/**
* Checks if the linked issue is a Zenhub epic.
*/
export async function isIssueLinkedToEpic(
Copy link
Member

Choose a reason for hiding this comment

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

Perhaps the naming should be different, as this sounds like you are checking that the given issue is linked to the epic, but in fact, you are checking that the actual given issue is the epic.

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

Labels

adhoc Ad-hoc unplanned task added during the sprint. t-discovery Issues owned by the discovery team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants