-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Split up astconv.rs
#67418
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
Comments
I'll do it, although I am not familiar with the If it is a problem that I am not familiar with the topic, I will back down, no worries. |
That's awesome. Please wait until #67268 is merged though (you can subscribe to it so you see when it does get merged), because otherwise you'll just get loads of conflicts. |
If you are unsure about the module structure, you can first post a design suggestion here if you want. |
@TheSamsa You still working on this? |
Not quite, my other PR is not yet through and I thought I wait for it. But if you want to do it, just take it. |
I would like to claim this, if nobody minds |
@rustbot assign @Areredify It's all yours! |
@rustbot release-assignment |
I'd like to work in this if @Areredify is not on it anymore @Dylan-DPC |
@CohenArthur generally it's fine if it's 3 months and there is no pr from the author. you can start working on it |
Gotcha, thanks |
@rustbot claim |
That file (https://github.com/rust-lang/rust/blob/dab390683196e31116cb61ccf3eec1a4af729def/src/librustc_typeck/astconv.rs) has almost 3k LOC and just 3 lines of documentation on the file (methods and types are documented though). I'm entirely unsure how to process that file. I think splitting it up into submodules (e.g. start by extracting out all functions just handling error reporting into a separate module).
Some of this work can be just mechanical splitting up, but some parts may require you to start understanding at least some of what's going on in the file in order to find a good module structure.
cc @Centril
The text was updated successfully, but these errors were encountered: