[PR #147] [MERGED] Fix path separator in gradlew.bat for Windows compatibility #201

New issue

Closed

opened 2026-02-27 19:26:45 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 19:26:45 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Lambada10/SongSync/pull/147
Author: @peter9811
Created: 2/22/2025
Status: ✅ Merged
Merged: 5/14/2025
Merged by: @Lambada10

Base: master ← Head: master

📝 Commits (6)

ad8d585 Fix path separator in gradlew.bat and update comments for clarity
a62eb22 Enhance code documentation and add utility functions for file handling and version retrieval
24a5650 Add documentation and fix button click handler in FailedDialogue composable
a44f5fa Merge remote-tracking branch 'upstream/master'
9f11118 Update README and Gradle scripts
640aa18 build: update gradle configuration and add dependencies

📊 Changes

27 files changed (+458 additions, -353 deletions)

View changed files

📝 .github/ISSUE_TEMPLATE/bug_report.md (+7 -6)
📝 .github/ISSUE_TEMPLATE/feature_request.md (+2 -3)
📝 .github/workflows/ci.yml (+53 -54)
➕ .vscode/settings.json (+6 -0)
📝 LICENSE (+165 -165)
📝 README.md (+18 -10)
📝 app/build.gradle.kts (+7 -1)
📝 app/src/main/AndroidManifest.xml (+7 -3)
📝 app/src/main/java/pl/lambada/songsync/data/remote/PaxMusicHelper.kt (+23 -11)
📝 app/src/main/java/pl/lambada/songsync/data/remote/UpdateService.kt (+12 -4)
📝 app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/others/AppleAPI.kt (+13 -1)
📝 app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/others/MusixmatchAPI.kt (+3 -3)
📝 app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/others/NeteaseAPI.kt (+5 -6)
📝 app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/others/QQMusicAPI.kt (+4 -8)
📝 app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/spotify/SpotifyAPI.kt (+4 -2)
📝 app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/spotify/SpotifyLyricsAPI.kt (+1 -1)
📝 app/src/main/java/pl/lambada/songsync/domain/model/SongInfo.kt (+9 -0)
📝 app/src/main/java/pl/lambada/songsync/domain/model/Sort.kt (+9 -0)
📝 app/src/main/java/pl/lambada/songsync/ui/screens/lyricsFetch/components/FailedDialogue.kt (+8 -3)
📝 app/src/main/java/pl/lambada/songsync/util/LyricsUtils.kt (+9 -5)

...and 7 more files

📄 Description

Correct the path separator in the gradlew.bat file and enhance comments for better clarity.

_{🔄 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/Lambada10/SongSync/pull/147 **Author:** [@peter9811](https://github.com/peter9811) **Created:** 2/22/2025 **Status:** ✅ Merged **Merged:** 5/14/2025 **Merged by:** [@Lambada10](https://github.com/Lambada10) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (6) - [`ad8d585`](https://github.com/Lambada10/SongSync/commit/ad8d58555ff0d01786cebd17dde02489107485da) Fix path separator in gradlew.bat and update comments for clarity - [`a62eb22`](https://github.com/Lambada10/SongSync/commit/a62eb228b14b98bbae3b12a18dcd0e51125ff005) Enhance code documentation and add utility functions for file handling and version retrieval - [`24a5650`](https://github.com/Lambada10/SongSync/commit/24a56506b60ed86d75944028265481fbbec13fc1) Add documentation and fix button click handler in FailedDialogue composable - [`a44f5fa`](https://github.com/Lambada10/SongSync/commit/a44f5fa38596019da97c863f937d6d78d2117b0b) Merge remote-tracking branch 'upstream/master' - [`9f11118`](https://github.com/Lambada10/SongSync/commit/9f11118ef981b0a288cd59bb343d79a8a74c9ea3) Update README and Gradle scripts - [`640aa18`](https://github.com/Lambada10/SongSync/commit/640aa18c49b024fddc3aff81849bf712d48b092f) build: update gradle configuration and add dependencies ### 📊 Changes **27 files changed** (+458 additions, -353 deletions) <details> <summary>View changed files</summary> 📝 `.github/ISSUE_TEMPLATE/bug_report.md` (+7 -6) 📝 `.github/ISSUE_TEMPLATE/feature_request.md` (+2 -3) 📝 `.github/workflows/ci.yml` (+53 -54) ➕ `.vscode/settings.json` (+6 -0) 📝 `LICENSE` (+165 -165) 📝 `README.md` (+18 -10) 📝 `app/build.gradle.kts` (+7 -1) 📝 `app/src/main/AndroidManifest.xml` (+7 -3) 📝 `app/src/main/java/pl/lambada/songsync/data/remote/PaxMusicHelper.kt` (+23 -11) 📝 `app/src/main/java/pl/lambada/songsync/data/remote/UpdateService.kt` (+12 -4) 📝 `app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/others/AppleAPI.kt` (+13 -1) 📝 `app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/others/MusixmatchAPI.kt` (+3 -3) 📝 `app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/others/NeteaseAPI.kt` (+5 -6) 📝 `app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/others/QQMusicAPI.kt` (+4 -8) 📝 `app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/spotify/SpotifyAPI.kt` (+4 -2) 📝 `app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/spotify/SpotifyLyricsAPI.kt` (+1 -1) 📝 `app/src/main/java/pl/lambada/songsync/domain/model/SongInfo.kt` (+9 -0) 📝 `app/src/main/java/pl/lambada/songsync/domain/model/Sort.kt` (+9 -0) 📝 `app/src/main/java/pl/lambada/songsync/ui/screens/lyricsFetch/components/FailedDialogue.kt` (+8 -3) 📝 `app/src/main/java/pl/lambada/songsync/util/LyricsUtils.kt` (+9 -5) _...and 7 more files_ </details> ### 📄 Description Correct the path separator in the gradlew.bat file and enhance comments for better clarity. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>