Skip to content

Labeling refactor/followups#3456

Closed
josephperrott wants to merge 7 commits intoangular:mainfrom
josephperrott:issue-labeling-follow-ups
Closed

Labeling refactor/followups#3456
josephperrott wants to merge 7 commits intoangular:mainfrom
josephperrott:issue-labeling-follow-ups

Conversation

@josephperrott
Copy link
Member

@josephperrott josephperrott commented Feb 27, 2026

No description provided.

@josephperrott josephperrott changed the title issue labeling follow ups Labeling refactor/followups Feb 27, 2026
Copy link
Contributor

@thePunderWoman thePunderWoman left a comment

Choose a reason for hiding this comment

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

LGTM

…base class

Extracts the common logic used by IssueLabeling and PullRequestLabeling into a
shared Labeling base class. This simplifies code duplication and unifies how
labels are added, removed, and authentication is handled with Octokit.
@josephperrott josephperrott force-pushed the issue-labeling-follow-ups branch from 242742d to 7d50193 Compare February 27, 2026 20:11
@josephperrott josephperrott added the action: merge The PR is ready for merge by the caretaker label Mar 2, 2026
@josephperrott josephperrott force-pushed the issue-labeling-follow-ups branch from a78f4c5 to 1217e76 Compare March 2, 2026 15:04
@josephperrott
Copy link
Member Author

This PR was merged into the repository. The changes were merged into the following branches:

josephperrott added a commit that referenced this pull request Mar 2, 2026
josephperrott added a commit that referenced this pull request Mar 2, 2026
josephperrott added a commit that referenced this pull request Mar 2, 2026
josephperrott added a commit that referenced this pull request Mar 2, 2026
josephperrott added a commit that referenced this pull request Mar 2, 2026
…base class (#3456)

Extracts the common logic used by IssueLabeling and PullRequestLabeling into a
shared Labeling base class. This simplifies code duplication and unifies how
labels are added, removed, and authentication is handled with Octokit.

PR Close #3456
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants