close
Skip to content

ci(release): squeeze extra blank lines in docs go.mod#799

Merged
ericcurtin merged 1 commit intodocker:mainfrom
doringeman:squeeze-docs-go-mod
Mar 27, 2026
Merged

ci(release): squeeze extra blank lines in docs go.mod#799
ericcurtin merged 1 commit intodocker:mainfrom
doringeman:squeeze-docs-go-mod

Conversation

@doringeman
Copy link
Copy Markdown
Contributor

Signed-off-by: Dorin Geman <dorin.geman@docker.com>
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

Copy link
Copy Markdown
Contributor

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

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

Hey - I've left some high level feedback:

  • Since you’re already streaming go.mod through awk, consider folding the blank-line squeezing into that awk script instead of adding a separate cat -s pass and second temp-file/mv cycle.
Prompt for AI Agents
Please address the comments from this code review:

## Overall Comments
- Since you’re already streaming go.mod through awk, consider folding the blank-line squeezing into that awk script instead of adding a separate `cat -s` pass and second temp-file/mv cycle.

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@ericcurtin ericcurtin merged commit f9f2e8b into docker:main Mar 27, 2026
7 checks passed
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.

3 participants