Skip to content

Formatted more masterfiles subdirectories with cfengine format#3179

Merged
olehermanse merged 5 commits into
cfengine:masterfrom
olehermanse:medium
Jun 26, 2026
Merged

Formatted more masterfiles subdirectories with cfengine format#3179
olehermanse merged 5 commits into
cfengine:masterfrom
olehermanse:medium

Conversation

@olehermanse

Copy link
Copy Markdown
Member
  • cfengine format ./cfe_internal/
  • cfengine format ./lib/
  • cfengine format ./tests/acceptance/30_custom_promise_types/
  • cfengine format ./tests/acceptance/lib/

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>
Signed-off-by: Ole Herman Schumacher Elgesem <ole@northern.tech>
@olehermanse

Copy link
Copy Markdown
Member Author

@cf-bottom jenkins, please

@olehermanse olehermanse marked this pull request as ready for review June 25, 2026 13:14
@olehermanse

Copy link
Copy Markdown
Member Author

@nickanderson @craigcomstock this one basically includes the last directories except packages tests (which have very large policy files).

@olehermanse olehermanse requested a review from larsewi June 25, 2026 13:18

@larsewi larsewi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

One way to preserve inline comment could be to have it as a different token. For example having it resolve to an inline_comment if it's not preceded by only whitespace.

Comment thread cfe_internal/enterprise/federation/federation.cf
Comment thread cfe_internal/update/update_policy.cf Outdated
Comment thread cfe_internal/update/windows_unattended_upgrade.cf Outdated
Comment thread lib/commands.cf
Comment thread lib/commands.cf
Comment thread lib/commands.cf
Comment thread lib/commands.cf
Comment thread lib/commands.cf
Comment thread lib/commands.cf
Comment thread lib/files.cf
Signed-off-by: Ole Herman Schumacher Elgesem <ole@northern.tech>
@olehermanse olehermanse requested a review from larsewi June 25, 2026 15:02
@cfengine cfengine deleted a comment from cf-bottom Jun 25, 2026
@cf-bottom

Copy link
Copy Markdown

@olehermanse

Copy link
Copy Markdown
Member Author

Test failures / flakes look unrelated. Known issues.

@olehermanse olehermanse merged commit 5847b69 into cfengine:master Jun 26, 2026
39 of 40 checks passed
@olehermanse olehermanse deleted the medium branch June 26, 2026 13:28
@olehermanse

Copy link
Copy Markdown
Member Author

Backports: #3193

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