Skip to content

feat: colored diff for fmt command #5652

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
Merged
merged 1 commit into from
Apr 4, 2025
Merged

feat: colored diff for fmt command #5652

merged 1 commit into from
Apr 4, 2025

Conversation

ldez
Copy link
Member

@ldez ldez commented Mar 28, 2025

--diff

Screenshot_20250328_210820

--diff-colored

Screenshot_20250328_210752

@ldez ldez added enhancement New feature or improvement topic: cosmetic changes contain cosmetic improvements area: formatters/fmt labels Mar 28, 2025
@ldez ldez added this to the v2-unreleased milestone Mar 28, 2025
@ldez ldez added the blocked: until next minor Will be a part of the next minor release label Mar 28, 2025
@ldez ldez removed this from the v2-unreleased milestone Mar 28, 2025
@ldez ldez force-pushed the feat/fmt-diff-colors branch from 30366b5 to c2a888f Compare April 4, 2025 00:36
@ldez ldez removed the blocked: until next minor Will be a part of the next minor release label Apr 4, 2025
@ldez ldez added this to the v2-unreleased milestone Apr 4, 2025
@ldez ldez merged commit 2edb46d into golangci:main Apr 4, 2025
19 checks passed
@ldez ldez deleted the feat/fmt-diff-colors branch April 4, 2025 00:56
@ldez ldez modified the milestones: v2-unreleased, v2.1 Apr 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ccoVeille ccoVeille ccoVeille left review comments

@alexandear alexandear alexandear approved these changes

@bombsimon bombsimon bombsimon approved these changes

Assignees
No one assigned
Labels
area: formatters/fmt enhancement New feature or improvement topic: cosmetic changes contain cosmetic improvements
Projects
None yet
Milestone
v2.1
Development

Successfully merging this pull request may close these issues.
4 participants
@ldez @alexandear @bombsimon @ccoVeille