Skip to content

Conversation

mariannegoldin
Copy link
Contributor

Fixing some formatting retroactively

@mariannegoldin mariannegoldin requested review from bdillo and bnchi August 21, 2025 03:02
@bnchi
Copy link
Contributor

bnchi commented Aug 21, 2025

Thank you @mariannegoldin looks good!

I was discussing with Ben on how can we eliminate these subtle mistakes, and was wondering if having different scripts for linting/formatting/collecting-events make sense, IMO we need to have all of these in one CLI program, and we can eventually maintain that single program or add any other tools that helps us in the process.

I would love to work on an issue that helps in moving the Python script to Rust and merge it with the rest of our beta tooling so that we can make this a bit more usable, and maybe create an issue to track the progress and help with any work that's still needed on the linter/merger scripts.

What do you think @bdillo

@bdillo
Copy link
Contributor

bdillo commented Aug 22, 2025

Thank you @mariannegoldin looks good!

I was discussing with Ben on how can we eliminate these subtle mistakes, and was wondering if having different scripts for linting/formatting/collecting-events make sense, IMO we need to have all of these in one CLI program, and we can eventually maintain that single program or add any other tools that helps us in the process.

I would love to work on an issue that helps in moving the Python script to Rust and merge it with the rest of our beta tooling so that we can make this a bit more usable, and maybe create an issue to track the progress and help with any work that's still needed on the linter/merger scripts.

What do you think @bdillo

I agree, it would be much nicer in the long term to just have everything in rust in general. I should be able to look at finishing the linter/new event merger thing soon and getting that in a working state, then we can see about moving the meetup api stuff over there as well.

@bdillo bdillo merged commit 2a265e6 into rust-lang:master Aug 22, 2025
1 check passed
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