Skip to content

chore(formatting): Update Astyle to 3.4#500

Merged
thomwiggers merged 3 commits intomasterfrom
update-astyle
Sep 21, 2023
Merged

chore(formatting): Update Astyle to 3.4#500
thomwiggers merged 3 commits intomasterfrom
update-astyle

Conversation

@thomwiggers
Copy link
Copy Markdown
Member

Seems that Astyle has started receiving maintenance again, with several releases in 2023. This PR runs Astyle 3.4's formatter over the whole repo. Aside from a lot of empty lines that got deleted, it seems that we have a few minor improvements in indentation across the repo.

@thomwiggers
Copy link
Copy Markdown
Member Author

Okay this is currently failing because of the astyle in the test images. I should probably update the containers to 3.4 as well...

dstebila
dstebila previously approved these changes Sep 12, 2023
@dstebila
Copy link
Copy Markdown
Member

I read the changes in the Python code but didn't look at any of the resulting changes in the .c or .h files, nor did I browse through the CI failures.

@thomwiggers
Copy link
Copy Markdown
Member Author

I do need to fix something in CI, please hold :)

@thomwiggers thomwiggers changed the title Update Astyle to 3.4 chore(formatting): Update Astyle to 3.4 Sep 21, 2023
@thomwiggers thomwiggers merged commit 719b509 into master Sep 21, 2023
@thomwiggers thomwiggers deleted the update-astyle branch September 21, 2023 12:22
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.

2 participants