Skip to content

cfengine format ./examples/#6204

Merged
olehermanse merged 3 commits into
cfengine:masterfrom
olehermanse:examples_two
Jul 1, 2026
Merged

cfengine format ./examples/#6204
olehermanse merged 3 commits into
cfengine:masterfrom
olehermanse:examples_two

Conversation

@olehermanse

@olehermanse olehermanse commented Jul 1, 2026

Copy link
Copy Markdown
Member

2nd try of #6197.

This time with preserving sameline comments.

Signed-off-by: Ole Herman Schumacher Elgesem <ole@northern.tech>
Signed-off-by: Ole Herman Schumacher Elgesem <ole@northern.tech>
Signed-off-by: Ole Herman Schumacher Elgesem <ole@northern.tech>
@olehermanse olehermanse marked this pull request as ready for review July 1, 2026 00:35
@olehermanse

Copy link
Copy Markdown
Member Author

@cf-bottom jenkins, please

@cf-bottom

Copy link
Copy Markdown

@nickanderson nickanderson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

There are some cases where long trailing comments were manually wrapped and vertically aligned with the associated trailing comment. It would be nice if we could keep that. Visually I find it much easier for scanning.

Comment thread examples/abort.cf
Comment thread examples/edit.empty_before_use.cf
Comment thread examples/edit.filename.cf
@olehermanse olehermanse merged commit cadb6b8 into cfengine:master Jul 1, 2026
43 checks passed
@olehermanse olehermanse deleted the examples_two branch July 1, 2026 16:28
@olehermanse

Copy link
Copy Markdown
Member Author

3.27 backport: #6218

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants