Skip to content

fix(github-actions): fix issue with milestones not applying#3628

Open
thePunderWoman wants to merge 1 commit intoangular:mainfrom
thePunderWoman:label-fix
Open

fix(github-actions): fix issue with milestones not applying#3628
thePunderWoman wants to merge 1 commit intoangular:mainfrom
thePunderWoman:label-fix

Conversation

@thePunderWoman
Copy link
Copy Markdown
Contributor

Milestones have not been set properly with the issue triage github action. It is likely due to the api response happening too quickly after labels were applied, and not yet including them. So this manually accounts for the labels we added.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a mechanism to track labels added during the current execution of the issue labeling action, ensuring that subsequent logic accounts for labels that might not yet be reflected in the GitHub API response due to latency. The feedback suggests improving encapsulation by making the addedLabels property private and centralizing the tracking logic by overriding the addLabel method, which would also ensure that all added labels are consistently recorded.

Comment thread github-actions/labeling/issue/lib/issue-labeling.ts Outdated
Comment thread github-actions/labeling/issue/lib/issue-labeling.ts Outdated
Milestones have not been set properly with the issue triage github action. It is likely due to the api response happening too quickly after labels were applied, and not yet including them. So this manually accounts for the labels we added.
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.

1 participant