ryanmcgrath / cacao

Rust bindings for AppKit (macOS) and UIKit (iOS/tvOS). Experimental, but working!
MIT License
1.8k stars 65 forks source link

Run `cargo fmt` #29

Closed madsmtm closed 1 year ago

madsmtm commented 1 year ago

Motivation: I find when reading code that it helps me quite a lot if it is formatted consistently.

I set the line width to 130 to keep the diff a bit smaller, if there's something else you don't quite like, I'm fine with tweaking the options.

Not really sure how to get this merged with the cleanest diff in the end, probably best of you merged the airyx-appkit-uikit-features branch first.

(Getting started on properly contributing to this crate, yay!)

ryanmcgrath commented 1 year ago

Heh, yeah… you probably noticed I have a distinct (ish) style I use when I write code. I don’t mind if people want standard cargo formatting though.

I’m out of town until Sunday evening but should be able to look at then or early this week. :)

Thanks!

On Sat, Jul 9, 2022 at 18:09, Mads Marquart @.***> wrote:

Motivation: I find when reading code that it helps me quite a lot if it is formatted consistently.

I set the line width to 130 to keep the diff a bit smaller, if there's something else you don't quite like, I'm fine with tweaking the options.

Not really sure how to get this merged with the cleanest diff in the end, probably best of you merged the airyx-appkit-uikit-features branch first.

(Getting started on properly contributing to this crate, yay!)


You can view, comment on, or merge this pull request online at:

https://github.com/ryanmcgrath/cacao/pull/29

Commit Summary

File Changes

(175 files)

Patch Links:

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.Message ID: @.***>

madsmtm commented 1 year ago

I tweaked a few more options to better match your "distinct (ish) style" 😉 - some of these require using cargo +nightly fmt though

ryanmcgrath commented 1 year ago

trunk should be up to date regarding any of my changes now - apologies if you got any notifications saying this was closed, it looks like that happened when I deleted the other branch... I restored it and reopened but feel free to reissue the PR if easier.

I'd be down to look at getting this merged, but I'll defer to you if you think it's ready to hit that "ready to review" button. Thanks again for moving on all this - super cool work!

madsmtm commented 1 year ago

Wonderful, I rebased so you can take a look now, each commit by itself should hopefully be fairly easy to verify