Skip to content

Small refactorings#87

Merged
olehermanse merged 5 commits intocfengine:mainfrom
olehermanse:improve
Apr 24, 2026
Merged

Small refactorings#87
olehermanse merged 5 commits intocfengine:mainfrom
olehermanse:improve

Conversation

@olehermanse
Copy link
Copy Markdown
Member

@olehermanse olehermanse commented Apr 22, 2026 •
edited
Loading

  • format.py: Made autoformat into an internal function _autoformat
  • Removed excessive unit testing of internal functions
  • format.py: Made can_single_line_promise internal
  • format.py: Made stringify internal
  • format.py: Made attempt_split_attribute internal

@olehermanse
format.py: Made autoformat into an internal function _autoformat
1b5edcf 
Signed-off-by: Ole Herman Schumacher Elgesem <ole@northern.tech>
@olehermanse
Removed excessive unit testing of internal functions
4165495 
Signed-off-by: Ole Herman Schumacher Elgesem <ole@northern.tech>
@olehermanse
format.py: Made can_single_line_promise internal
b843646 
Signed-off-by: Ole Herman Schumacher Elgesem <ole@northern.tech>
@olehermanse
format.py: Made stringify internal
4faadc0 
Signed-off-by: Ole Herman Schumacher Elgesem <ole@northern.tech>
@olehermanse
format.py: Made attempt_split_attribute internal
291498f 
Signed-off-by: Ole Herman Schumacher Elgesem <ole@northern.tech>
@olehermanse olehermanse changed the title improve Small refactorings Apr 23, 2026
@olehermanse olehermanse marked this pull request as ready for review April 23, 2026 12:30
@olehermanse olehermanse merged commit 4df615d into cfengine:main Apr 24, 2026
5 checks passed
@olehermanse olehermanse deleted the improve branch April 24, 2026 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@larsewi larsewi larsewi approved these changes

@craigcomstock craigcomstock Awaiting requested review from craigcomstock

@aleksandrychev aleksandrychev Awaiting requested review from aleksandrychev

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@olehermanse @larsewi