[PR #188] [MERGED] Replace html links with intra-documentation links #298

Closed
opened 2026-02-27 20:24:09 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ramsayleung/rspotify/pull/188
Author: @ramsayleung
Created: 2/17/2021
Status: Merged
Merged: 2/19/2021
Merged by: @marioortizmanero

Base: masterHead: ramsay_easier_linking_doc


📝 Commits (4)

📊 Changes

4 files changed (+20 additions, -21 deletions)

View changed files

📝 CHANGELOG.md (+1 -0)
📝 src/client.rs (+4 -4)
📝 src/lib.rs (+14 -16)
📝 src/oauth2.rs (+1 -1)

📄 Description

Description

Use direct intra-documentation links instead of html links we currently use, which make them much easier to read and write, tring to fix #187

Motivation and Context

Make rspotify easier to maintain

Dependencies

Type of change

Please delete options that are not relevant.

  • This change requires a documentation update

How Has This Been Tested?

Generate doc with cargo doc --features env-file,cli, then click the links in generated html doc by hand to check if it's ok.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/ramsayleung/rspotify/pull/188 **Author:** [@ramsayleung](https://github.com/ramsayleung) **Created:** 2/17/2021 **Status:** ✅ Merged **Merged:** 2/19/2021 **Merged by:** [@marioortizmanero](https://github.com/marioortizmanero) **Base:** `master` ← **Head:** `ramsay_easier_linking_doc` --- ### 📝 Commits (4) - [`ae08f15`](https://github.com/ramsayleung/rspotify/commit/ae08f15483dbd12de4f78a2ee99d2f08b07ee808) replace html linking with rustdoc infra linking - [`78d61a8`](https://github.com/ramsayleung/rspotify/commit/78d61a82647221c2a7fccb6de2d304f5bbb29364) fix raw links - [`21c409b`](https://github.com/ramsayleung/rspotify/commit/21c409b246aafc0801fb75d88699373639c3dede) Update CHANGELOG - [`9ccabfb`](https://github.com/ramsayleung/rspotify/commit/9ccabfbb0f529ae51591720cb3ad87006a8abf49) Fix missing html links ### 📊 Changes **4 files changed** (+20 additions, -21 deletions) <details> <summary>View changed files</summary> 📝 `CHANGELOG.md` (+1 -0) 📝 `src/client.rs` (+4 -4) 📝 `src/lib.rs` (+14 -16) 📝 `src/oauth2.rs` (+1 -1) </details> ### 📄 Description ## Description Use direct intra-documentation links instead of html links we currently use, which make them much easier to read and write, tring to fix #187 ## Motivation and Context Make rspotify easier to maintain ## Dependencies ## Type of change Please delete options that are not relevant. - [x] This change requires a documentation update ## How Has This Been Tested? Generate doc with `cargo doc --features env-file,cli`, then click the links in generated html doc by hand to check if it's ok. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-27 20:24:09 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/rspotify#298
No description provided.