Skip to content
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

Support renaming re-exports, glob re-exports, and type aliases used as renames. #34

Merged
merged 4 commits into from
Jan 27, 2023

Conversation

obi1kenobi
Copy link
Owner

@obi1kenobi obi1kenobi commented Jan 14, 2023

Best reviewed one commit at a time:

  • The first commit adds a test suite with many tricky edge cases, and logic for handling renames and glob re-exports.
  • The second commit handles cyclic import paths, which before this PR would cause a stack overflow crash.
  • The third commit handles type aliases used as renames: type Foo = Bar or type Foo<T> = Bar<T>, as opposed to type Foo = Bar<u64> which is not just a rename but also fills generic parameter slots.
  • The last commit makes it easier to match the Rust toolchain version to the expected rustdoc version on the branch.

@obi1kenobi obi1kenobi marked this pull request as draft January 14, 2023 01:35
@obi1kenobi obi1kenobi force-pushed the imports_test_suite branch 8 times, most recently from d68682d to 599b107 Compare January 17, 2023 17:27
@obi1kenobi obi1kenobi force-pushed the imports_test_suite branch 4 times, most recently from 86901da to 4fffdeb Compare January 26, 2023 05:39
@obi1kenobi obi1kenobi force-pushed the imports_test_suite branch 6 times, most recently from 538bbbd to 349bbcd Compare January 26, 2023 06:20
@obi1kenobi obi1kenobi marked this pull request as ready for review January 26, 2023 06:36
@obi1kenobi obi1kenobi enabled auto-merge (squash) January 27, 2023 15:04
@obi1kenobi obi1kenobi disabled auto-merge January 27, 2023 15:06
@obi1kenobi obi1kenobi merged commit 5aeaae1 into rustdoc-v24 Jan 27, 2023
@obi1kenobi obi1kenobi deleted the imports_test_suite branch January 27, 2023 15:06
obi1kenobi added a commit that referenced this pull request Jan 27, 2023
…s renames. (#34)

* Implement `pub use` and glob re-exports.

* Reasonably handle import paths with cycles rather than crashing.

* Handle `pub type` uses equivalent to `pub use` as mere re-exports.

* Switch tests to nightly since that's where rustdoc v24 is.
obi1kenobi added a commit that referenced this pull request Jan 27, 2023
…s renames. (#34)

* Implement `pub use` and glob re-exports.

* Reasonably handle import paths with cycles rather than crashing.

* Handle `pub type` uses equivalent to `pub use` as mere re-exports.

* Switch tests to nightly since that's where rustdoc v24 is.
obi1kenobi added a commit that referenced this pull request Jan 27, 2023
…s renames. (#34) (#39)

* Implement `pub use` and glob re-exports.

* Reasonably handle import paths with cycles rather than crashing.

* Handle `pub type` uses equivalent to `pub use` as mere re-exports.

* Switch tests to nightly since that's where rustdoc v24 is.
obi1kenobi added a commit that referenced this pull request Jan 27, 2023
…s renames. (#34) (#38)

* Implement `pub use` and glob re-exports.

* Reasonably handle import paths with cycles rather than crashing.

* Handle `pub type` uses equivalent to `pub use` as mere re-exports.

* Switch tests to nightly since that's where rustdoc v24 is.
obi1kenobi added a commit that referenced this pull request Jan 27, 2023
…s renames. (#34)

* Implement `pub use` and glob re-exports.

* Reasonably handle import paths with cycles rather than crashing.

* Handle `pub type` uses equivalent to `pub use` as mere re-exports.

* Switch tests to nightly since that's where rustdoc v24 is.
obi1kenobi added a commit that referenced this pull request Jan 27, 2023
…s renames. (#34) (#47)

* Implement `pub use` and glob re-exports.

* Reasonably handle import paths with cycles rather than crashing.

* Handle `pub type` uses equivalent to `pub use` as mere re-exports.

* Switch tests to nightly since that's where rustdoc v24 is.
This was referenced Apr 14, 2023
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.

1 participant