Skip to content

Adding #4303 to stable #5180

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 4 commits into from
Closed

Adding #4303 to stable #5180

wants to merge 4 commits into from

Conversation

murchu27
Copy link
Contributor

@ayazhafiz originally created #4303 for v2.0, and it never reached stable (see #5173). This PR cherry-picks the commits from #4303, with a few merge conflicts in src/items.rs resolved.

I only had time to do the cherry-pick, I haven't tested this yet. I'm not sure if anything has changed in the codebase since #4303 that would cause this to no longer work.

I'll get around to testing it soon, just wanted to open the PR now in case someone else got time before me!

Cherry-picked from ayazhafiz, branch i/2868. That commit was originally
implemented on the 2.0 branch, while this commit adds it to stable.

Original commit message:

> Adds the option for
>
> - Functions
> - Control flow blocks
> - Expression statements (i.e. `let x = { ... };`)
> - Traits
> - Trait Impls
> - Mods
> - Foreign Mods
>
> If I am missing a kind of item, let me know :)
>
> Closes rust-lang#2868
Cherry-picked from ayazhafiz, branch i/2868. That commit was originally
implemented on the 2.0 branch, while this commit adds it to stable.
Cherry-picked from ayazhafiz, branch i/2868. That commit was originally
implemented on the 2.0 branch, while this commit adds it to stable.
Cherry-picked from ayazhafiz, branch i/2868. That commit was originally
implemented on the 2.0 branch, while this commit adds it to stable.
@murchu27
Copy link
Contributor Author

Yikes, I definitely need to do more with this. I'll convert it to a draft for now

@murchu27 murchu27 marked this pull request as draft January 21, 2022 00:48
@calebcartwright
Copy link
Member

Thanks, though I'm going to close this to reduce notification noise

@ayazhafiz
Copy link
Contributor

@murchu27 if you want some help bringing this back, feel free to email me (it’s in my bio). I haven’t worked on this project in a while but roughly remember getting around it and how this was done, etc.

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