Fix duplicate link references in README #7020
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Markdown can't handle multiple reference links with same name. i.e.
The link on
Now how about this link
will point to arrow-rs instead of datafusion.Example of how that's affecting current README:
For now I've decided to include the issue number as part of the text so the reference links have unique names.
Alternatives were either slightly change the text, like making the second one:
Which I didn't like as I feel we may fall into this issue again.
Another alternative would be to inline both:
Don't have a particular preference for that or current approach.