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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve labeler rules for kernel label #724

Merged
merged 2 commits into from Apr 28, 2023

Conversation

lemillermicrosoft
Copy link
Member

@lemillermicrosoft lemillermicrosoft commented Apr 28, 2023

Motivation and Context

#710 tried to enable the kernel tag but accidentally applied to too many changes.

Description

The best way to resolve this was specify directories under dotnet to be considered for kernel.

lemillermicrosoft#7
lemillermicrosoft#8
lemillermicrosoft#10
lemillermicrosoft#9 This one was missed before.

Contribution Checklist

@github-actions github-actions bot added the kernel Issues or pull requests impacting the core kernel label Apr 28, 2023
@lemillermicrosoft lemillermicrosoft added PR: ready for review All feedback addressed, ready for reviews PR: ready to merge PR has been approved by all reviewers, and is ready to merge. labels Apr 28, 2023
This commit updates the .github/labeler.yml file to better match the changes in the dotnet/src directory. It simplifies the rules for the 'kernel' label and adds a new rule for the 'SemanticKernel.MetaPackage' project. It also removes some redundant patterns that are already covered by the '**/*' glob. This will help to automatically assign labels to pull requests that affect the kernel or .NET projects.
@lemillermicrosoft lemillermicrosoft merged commit c4e1449 into microsoft:main Apr 28, 2023
11 checks passed
dluc pushed a commit that referenced this pull request Apr 29, 2023
### Motivation and Context
#710 tried to enable the `kernel` tag but accidentally applied to too
many changes.

### Description
The best way to resolve this was specify directories under `dotnet` to
be considered for `kernel`.
 
 lemillermicrosoft#7
 lemillermicrosoft#8
 lemillermicrosoft#10
lemillermicrosoft#9 This one was
missed before.
dehoward pushed a commit to lemillermicrosoft/semantic-kernel that referenced this pull request Jun 1, 2023
### Motivation and Context
microsoft#710 tried to enable the `kernel` tag but accidentally applied to too
many changes.

### Description
The best way to resolve this was specify directories under `dotnet` to
be considered for `kernel`.
 
 #7
 #8
 #10
#9 This one was
missed before.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kernel Issues or pull requests impacting the core kernel PR: ready for review All feedback addressed, ready for reviews PR: ready to merge PR has been approved by all reviewers, and is ready to merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants