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.

Sign up for GitHub

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

Jump to bottom

ci: automate-labels for dependabot PRs #1481

Merged
merged 2 commits into from
Mar 13, 2023
Merged

ci: automate-labels for dependabot PRs #1481

merged 2 commits into from
Mar 13, 2023

Conversation

amityadav0
Copy link
Contributor

@amityadav0 amityadav0 commented Mar 13, 2023 •
edited by rootulp

Automate-labels for dependabot PRs

Closes #1464

@MSevey MSevey requested review from a team and staheri14 and removed request for a team March 13, 2023 19:51
rootulp
rootulp reviewed Mar 13, 2023
View reviewed changes
Copy link
Collaborator

@rootulp rootulp left a comment

Choose a reason for hiding this comment

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

LGTM after a comment change! Thanks for the contribution

.github/workflows/issue-label-automation.yml Outdated Show resolved Hide resolved
@rootulp rootulp changed the title Automate-labels for dependabot PRs ci: automate-labels for dependabot PRs Mar 13, 2023
staheri14
staheri14 reviewed Mar 13, 2023
View reviewed changes
Copy link
Contributor

@staheri14 staheri14 left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution, looks good to me too! though, I refer it to the code owners to give the final approval.

@rootulp rootulp mentioned this pull request Mar 13, 2023
Merged
@MSevey MSevey requested a review from a team March 13, 2023 20:33
@rootulp rootulp merged commit 30ab522 into celestiaorg:main Mar 13, 2023
rootulp added a commit that referenced this pull request Mar 13, 2023
@rootulp
chore: add Sanaz as NMT wrapper code owner (#1482)
424f361 
Inspired by
#1481 (review)
😆
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@staheri14 staheri14 staheri14 left review comments

@rootulp rootulp rootulp approved these changes

@evan-forbes evan-forbes Awaiting requested review from evan-forbes

Assignees

@amityadav0 amityadav0

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

ci: automate-labels fails for dependabot PRs
3 participants
@amityadav0 @rootulp @staheri14