Skip to content

refactor(@angular/cli): remove unused eslint-disable comment in git utility - #33763

Merged
alan-agius4 merged 1 commit into
angular:mainfrom
alan-agius4:clean-git-utility
Aug 5, 2026
Merged

refactor(@angular/cli): remove unused eslint-disable comment in git utility#33763
alan-agius4 merged 1 commit into
angular:mainfrom
alan-agius4:clean-git-utility

Conversation

@alan-agius4

Copy link
Copy Markdown
Collaborator

Remove unused // eslint-disable-line no-empty comment in checkCleanGit utility function.

@alan-agius4
alan-agius4 requested a review from clydin August 5, 2026 08:09
@alan-agius4 alan-agius4 added action: review The PR is still awaiting reviews from at least one requested reviewer target: patch This PR is targeted for the next patch release labels Aug 5, 2026

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

Copy link
Copy Markdown

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 removes an unnecessary eslint-disable-line comment from an empty catch block in the git utility file. There are no review comments, and I have no feedback to provide.

@alan-agius4
alan-agius4 requested a review from JeanMeche August 5, 2026 08:10
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Aug 5, 2026
@alan-agius4
alan-agius4 removed the request for review from clydin August 5, 2026 08:12
@alan-agius4
alan-agius4 merged commit a632631 into angular:main Aug 5, 2026
40 of 43 checks passed
@alan-agius4
alan-agius4 deleted the clean-git-utility branch August 5, 2026 08:12
@alan-agius4

Copy link
Copy Markdown
Collaborator Author

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

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 area: @angular/cli target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants