Skip to content

Conversation

liamdebeasi
Copy link
Contributor

@liamdebeasi liamdebeasi commented Dec 4, 2023

Issue number: N/A


What is the current behavior?

The labeler action is currently failing: https://github.com/ionic-team/ionic-framework/actions/runs/7090913880/job/19298918578?pr=28622. This is happening due to a breaking change in v5 of the action.

We currently pull from main for this action so we are now receiving v5 of the action:

- uses: actions/labeler@main

What is the new behavior?

  • This PR updates the action explicitly to v5 so we don't unexpectedly take on breaking changes
  • This PR also updates the labeler yaml file to account for the v5 breaking changes

Does this introduce a breaking change?

  • Yes
  • No

Other information

@liamdebeasi liamdebeasi changed the title Liamdebeasi patch 1 chore(ci): update labeler workflow for labeler@v5 Dec 4, 2023
@liamdebeasi

This comment was marked as outdated.

@liamdebeasi liamdebeasi closed this Dec 4, 2023
@liamdebeasi liamdebeasi reopened this Dec 4, 2023
@liamdebeasi
Copy link
Contributor Author

Looks like I may have implemented this correctly, but the labeler is pulling the wrong version of the YAML file: https://github.com/ionic-team/ionic-framework/actions/runs/7092087270/job/19302623106?pr=28627

@liamdebeasi liamdebeasi marked this pull request as ready for review December 4, 2023 21:26
@liamdebeasi liamdebeasi requested a review from a team as a code owner December 4, 2023 21:26
@liamdebeasi liamdebeasi requested review from brandyscarney and removed request for a team December 4, 2023 21:26
@liamdebeasi
Copy link
Contributor Author

Spoke with Sean and our current thinking is we need to merge this to see any changes in the state of the GitHub Actions job. Worst case is this continues to fail, but no one should be blocked on merging since it's not a required status check.

@liamdebeasi liamdebeasi added this pull request to the merge queue Dec 4, 2023
Merged via the queue into main with commit 4847488 Dec 4, 2023
@liamdebeasi liamdebeasi deleted the liamdebeasi-patch-1 branch December 4, 2023 21:42
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