Re: [dev-servo] Rustfmt now checked on CI

2018-11-08 Thread Manish Goregaokar
Directly run rustfmt on the file, with `rustfmt foo.rs`. you may need to install it with `rustup component add rustfmt` On Thu, Nov 8, 2018, 6:50 PM Avanthikaa Ravichandran We are working on the media crate for servo and we were asked to run the > rustfmt to format the code. However, in the

Re: [dev-servo] Rustfmt now checked on CI

2018-11-08 Thread Avanthikaa Ravichandran
We are working on the media crate for servo and we were asked to run the rustfmt to format the code. However, in the servo/media repo that we worked on, we cannot run the ./mach command. Are we expected to integrate it with the main servo repository? How do we run rustfmt on our file otherwise?

Re: [dev-servo] Rustfmt now checked on CI

2018-11-07 Thread Keith Yeung
Woohoo! Great work everyone! On Wed, Nov 7, 2018 at 11:22 AM Josh Bowman-Matthews wrote: > > Thank you for all your help in formatting the existing code and enabling > rustfmt in CI! I'm very pleased that we've adopted rustfmt and that we > now have consistent formatting with the rest of the Rust

Re: [dev-servo] Rustfmt now checked on CI

2018-11-07 Thread Josh Bowman-Matthews
Thank you for all your help in formatting the existing code and enabling rustfmt in CI! I'm very pleased that we've adopted rustfmt and that we now have consistent formatting with the rest of the Rust ecosystem. Cheers, Josh On 11/7/18 1:51 PM, Pyfisch wrote: Hi Everyone, Servo source code

[dev-servo] Rustfmt now checked on CI

2018-11-07 Thread Pyfisch
Hi Everyone, Servo source code is now formatted with rustfmt [1]. Style checking is part of ./mach test-tidy and enforced on CI. To automatically format your code run ./mach fmt before commiting. Import order was changed but ./mach fmt will fix any issues. Please ask any questions you may