Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CI] Fix permissions value for the PR Labeler, switch to a new labeler #1007

Merged

Conversation

akashchi
Copy link
Contributor

@akashchi akashchi commented Apr 12, 2023

The File Labeler Action has a bug when additional file types are specified (e.g. added, modified) so that the action fails to assign respective labels.

I have found the bug and created an issue in the repository.

Meanwhile, I forked the labeler and fixed the bug in my repository. The forked action is used in this PR until the issue is fixed in the parent repository.

The fixed action works.

@akashchi akashchi changed the title [CI] Fix permission value for the PR Labeler [WIP] [CI] Fix permission value for the PR Labeler Apr 12, 2023
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@akashchi akashchi changed the title [WIP] [CI] Fix permission value for the PR Labeler [WIP] [CI] Fix permissions value for the PR Labeler, switch to a new labeler Apr 13, 2023
@akashchi akashchi changed the title [WIP] [CI] Fix permissions value for the PR Labeler, switch to a new labeler [CI] Fix permissions value for the PR Labeler, switch to a new labeler Apr 13, 2023
@adrianboguszewski
Copy link
Contributor

@akashchi, I can see it's failing. Is it ready to be merged?

@akashchi
Copy link
Contributor Author

@akashchi, I can see it's failing. Is it ready to be merged?

Yes, it is ready. It is failing due to insufficient permissions, the updated permissions will take effect once the updated .yml is in the parent repository.

@adrianboguszewski adrianboguszewski merged commit 5772647 into openvinotoolkit:main Apr 25, 2023
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.

3 participants