[PR #110] [MERGED] Reduce core dependencies #249

New issue

Closed

opened 2026-02-27 20:23:57 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 20:23:57 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/ramsayleung/rspotify/pull/110
Author: @ramsayleung
Created: 8/14/2020
Status: ✅ Merged
Merged: 8/28/2020
Merged by: @ramsayleung

Base: master ← Head: reduce-core-dependencies

📝 Commits (10+)

22e9ec8 Replace rand with getrandom.
c98a577 replacing rand with getrandom to reduce binary size and compilation time.
88c97ba Merge branch 'reduce-core-dependencies' of github.com:samrayleung/rspotify into reduce-core-dependencies
08ded2b Merge branch 'master' into reduce-core-dependencies
d36adb0 Change alphanum from global variable to a local variable, update description of
f6caa06 Change rand dependency into getrandom
09ee436 Moved static variable inside function
be15092 Optional webbrowser feature
2841d99 Optional dotenv feature
6d6cbdf Initial documentation work

📊 Changes

14 files changed (+219 additions, -88 deletions)

View changed files

📝 .github/workflows/ci.yml (+1 -0)
📝 CHANGELOG.md (+20 -1)
📝 Cargo.toml (+43 -12)
📝 examples/webapp/Cargo.toml (+2 -4)
📝 examples/webapp/src/main.rs (+11 -12)
📝 examples/with_refresh_token.rs (+0 -3)
📝 src/blocking/client.rs (+31 -10)
📝 src/blocking/oauth2.rs (+8 -3)
📝 src/blocking/util.rs (+14 -7)
📝 src/client.rs (+26 -8)
📝 src/lib.rs (+39 -14)
📝 src/oauth2.rs (+8 -3)
📝 src/util.rs (+14 -6)
📝 tests/test_with_oauth.rs (+2 -5)

📄 Description

Reducing core dependencies, check this issue https://github.com/ramsayleung/rspotify/issues/108 for more details:

replacing rand with a lightweight replacement getrandom
optional dotenv dependency
optional browser dependency
documentation

_{🔄 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/110 **Author:** [@ramsayleung](https://github.com/ramsayleung) **Created:** 8/14/2020 **Status:** ✅ Merged **Merged:** 8/28/2020 **Merged by:** [@ramsayleung](https://github.com/ramsayleung) **Base:** `master` ← **Head:** `reduce-core-dependencies` --- ### 📝 Commits (10+) - [`22e9ec8`](https://github.com/ramsayleung/rspotify/commit/22e9ec880bc57671abce47a175aa31e849894585) Replace rand with getrandom. - [`c98a577`](https://github.com/ramsayleung/rspotify/commit/c98a57770b42922510bc35bc17afdf67c27ddb90) replacing rand with getrandom to reduce binary size and compilation time. - [`88c97ba`](https://github.com/ramsayleung/rspotify/commit/88c97ba5c6e455b9fe72703521933ecca278d86c) Merge branch 'reduce-core-dependencies' of github.com:samrayleung/rspotify into reduce-core-dependencies - [`08ded2b`](https://github.com/ramsayleung/rspotify/commit/08ded2be56e9e9267354daf5c3854e23776f9eb7) Merge branch 'master' into reduce-core-dependencies - [`d36adb0`](https://github.com/ramsayleung/rspotify/commit/d36adb0b7974ee526d2f7e5fc0e7c09d8efbbce1) Change alphanum from global variable to a local variable, update description of - [`f6caa06`](https://github.com/ramsayleung/rspotify/commit/f6caa0696d328de58ee04f74db5089a683869a9f) Change rand dependency into getrandom - [`09ee436`](https://github.com/ramsayleung/rspotify/commit/09ee436574396cd77c3c08416a15e3933157e2fb) Moved static variable inside function - [`be15092`](https://github.com/ramsayleung/rspotify/commit/be15092e51bdc7771153cdab8390ed1b7a254807) Optional webbrowser feature - [`2841d99`](https://github.com/ramsayleung/rspotify/commit/2841d99404bc3c48c31d907e66be2e3170250f1f) Optional dotenv feature - [`6d6cbdf`](https://github.com/ramsayleung/rspotify/commit/6d6cbdfe86c0eccab9f854f30d6036381d6d36e1) Initial documentation work ### 📊 Changes **14 files changed** (+219 additions, -88 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/ci.yml` (+1 -0) 📝 `CHANGELOG.md` (+20 -1) 📝 `Cargo.toml` (+43 -12) 📝 `examples/webapp/Cargo.toml` (+2 -4) 📝 `examples/webapp/src/main.rs` (+11 -12) 📝 `examples/with_refresh_token.rs` (+0 -3) 📝 `src/blocking/client.rs` (+31 -10) 📝 `src/blocking/oauth2.rs` (+8 -3) 📝 `src/blocking/util.rs` (+14 -7) 📝 `src/client.rs` (+26 -8) 📝 `src/lib.rs` (+39 -14) 📝 `src/oauth2.rs` (+8 -3) 📝 `src/util.rs` (+14 -6) 📝 `tests/test_with_oauth.rs` (+2 -5) </details> ### 📄 Description Reducing core dependencies, check this issue https://github.com/ramsayleung/rspotify/issues/108 for more details: - [x] replacing `rand` with a lightweight replacement `getrandom` - [x] optional `dotenv` dependency - [x] optional `browser` dependency - [x] documentation --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>