-
Notifications
You must be signed in to change notification settings - Fork 256
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
Remove line links from ff/README #584
base: master
Are you sure you want to change the base?
Conversation
Line links are difficult to maintain (and currently most of them are wrong). Removing them and improving the documentation with same links.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, I agree this is a good change! Can you check what the links point to when compiled to docs via cargo doc?
hmm it translates into broken links both with prefixed |
Hm in that case there's three options: |
good point, going for (a) |
right, even with the latest commit I see how that breaks the readme.. |
ping. @Pratyush how about leaving the readme alone? |
Line links are difficult to maintain (and currently most of them are wrong). Removing them and improving the documentation with same links.