Skip to content

Latest arch uncrustify #7714

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 3 commits into from
Closed

Conversation

bill88t
Copy link

@bill88t bill88t commented Mar 11, 2023

Docs specify no uncrustify version.
Arch:latest provides version Uncrustify_d-0.75.1_f which produces the following during pre-commit run --all

Copy link
Collaborator

@microdev1 microdev1 left a comment

Choose a reason for hiding this comment

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

We are limited by the uncrustify version in MicroPython. Changing ours will complicate a merge from upstream.
The docs mention this version limitation.

@bill88t
Copy link
Author

bill88t commented Mar 11, 2023

Thanks and sorry for missing that.

@bill88t bill88t closed this Mar 11, 2023
@bill88t bill88t deleted the latest-uncrustify branch March 11, 2023 14:01
@tannewt
Copy link
Member

tannewt commented Mar 13, 2023

@bill88t I have a version of uncrustify 0.72 if you need it. I think we're also limited by what Ubuntu packages because that's what the CI uses.

@bill88t
Copy link
Author

bill88t commented Mar 13, 2023

Aur has a nice package named downgrade, it got me to 0.71 in the blink of an eye. Thanks for the offer though.

@tannewt
Copy link
Member

tannewt commented Mar 14, 2023

Thanks for the tip!

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