Skip to content

Conversation

@enkerewpo
Copy link
Member

No description provided.

enkerewpo added 4 commits May 5, 2025 14:33
…labels are added only when an issue number is present, improving debugging and reliability.
…ber, ensuring labels are applied correctly for pull requests.
…ng brace, ensuring proper execution of label assignment logic.
…ll request number and computed labels, improving debugging capabilities.
@github-actions github-actions bot added bug Something isn't working ci Github CI labels May 5, 2025
@enkerewpo enkerewpo requested a review from Copilot May 5, 2025 06:46
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 fixes crashes in the CI labeler workflows by ensuring that API calls to add labels are only made when there are labels present and by correctly referencing the pull request and issue numbers.

  • Updated pull request labeler to use pull_request number and added logging for debugging.
  • Updated issue labeler to use issue number from the payload and added logging for enhanced visibility.

Reviewed Changes

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

File Description
.github/workflows/labeler.yml Uses pull_request number and logs debugging details; adds a label existence check before API call.
.github/workflows/issue-labeler.yml Uses issue number from the payload and logs label details; adds a condition to verify label presence.

@enkerewpo enkerewpo changed the title ci. fixed pull request and issue labeller crashed on empty arrays ci. fixed pull request and issue labeler crashed on empty arrays May 5, 2025
@KouweiLee KouweiLee merged commit 8d7e4d2 into syswonder:dev May 5, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working ci Github CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants