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.

Sign up for GitHub

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

Jump to bottom

docs: reorder contributor steps, rustup is required for git hooks install to work smoothly #18828

Merged

jsirois merged 1 commit into pantsbuild:main from biancarosa:reorder-docs

Apr 24, 2023

Contributor

biancarosa commented Apr 24, 2023

While doing the contributor setup steps, I stopped for a second to read the rustup error when installing the git hooks.

I then saw that step 3 literally tells to install rustup.

It'd be nice to have those reodered in order to have a smoother start for new contributors.


          docs: reorder contributor steps, rustup is required for git hooks ins…

2d815ce

…tall to work smoothly

jsirois approved these changes

View reviewed changes

Member

jsirois left a comment

Thanks @biancarosa

jsirois added the category:documentation label

jsirois merged commit 15ff2c6 into pantsbuild:main

17 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment