Skip to content

Conversation

@yoshuawuyts
Copy link
Member

While authoring rust-lang/rust#88154 I realized that the codegen for the enum_generate_is_method assist currently generates invalid paths, and used snake case instead of spaces for the docs description. This fixes both issues. Thanks!

@Veykril
Copy link
Member

Veykril commented Aug 19, 2021

bors d+
needs a doc test update, running cargo test -p ide_assists will fix that up(not sure what the specific test name is that does the updating there)

@bors
Copy link
Contributor

bors bot commented Aug 19, 2021

✌️ yoshuawuyts can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

Previously the generated paths were invalid. This fixes that.
@yoshuawuyts yoshuawuyts force-pushed the fix-is_method-links-gen branch from 818485f to 586d6fc Compare August 19, 2021 11:31
@yoshuawuyts
Copy link
Member Author

bors r+

@bors
Copy link
Contributor

bors bot commented Aug 19, 2021

@bors bors bot merged commit f696fb5 into rust-lang:master Aug 19, 2021
@yoshuawuyts yoshuawuyts deleted the fix-is_method-links-gen branch August 19, 2021 11:39
@lnicola lnicola changed the title Fix codegen for is_method documentation fix: Fix codegen for is_method documentation Aug 19, 2021
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.

2 participants