Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Using placeholder rustdoc generation for current crate #351

Conversation

tonowak
Copy link
Collaborator

@tonowak tonowak commented Feb 4, 2023

This PR is stacked on #350.

It changes the logic of the tool. Instead of generating the rustdoc of the current crate directly, it uses the baseline's way of creating a placeholder project and running rustdoc on it.

@tonowak tonowak requested a review from obi1kenobi February 4, 2023 21:11
@tonowak
Copy link
Collaborator Author

tonowak commented Feb 4, 2023

This PR should fix #167, but I don't have an Apple I could use to check it. @obi1kenobi can you test whether this PR fixes the issue?

Copy link
Owner

@obi1kenobi obi1kenobi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good too, just the one nitpick.

@obi1kenobi
Copy link
Owner

This PR should fix #167, but I don't have an Apple I could use to check it. @obi1kenobi can you test whether this PR fixes the issue?

I don't have an Apple device either. Maybe we can ask the original reporter of #167 to try it once we release this.

@tonowak tonowak requested a review from obi1kenobi February 5, 2023 15:06
@obi1kenobi obi1kenobi merged commit 135cb33 into obi1kenobi:main Feb 5, 2023
@tonowak tonowak mentioned this pull request Feb 5, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants