Skip to content

Commit 2ad662b

Browse files
savitojssavitojs
committed
refactor: Improved User Experience
Signed-off-by: Savitoj Singh <savitojs@gmail.com>
1 parent 2402662 commit 2ad662b

File tree

15 files changed

+2297
-467
lines changed

15 files changed

+2297
-467
lines changed

‎.actrc‎

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
# Act configuration for local GitHub Actions testing
2+
# This file configures the container images and settings for act
3+
4+
# Use GitHub's runner images for better compatibility
5+
-P ubuntu-latest=catthehacker/ubuntu:act-latest
6+
-P ubuntu-22.04=catthehacker/ubuntu:act-22.04
7+
-P ubuntu-20.04=catthehacker/ubuntu:act-20.04
8+
9+
# Environment variables that might be needed
10+
--env GITHUB_TOKEN=fake_token_for_testing

‎.github/events/tag-push.json‎

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
{
2+
"ref": "refs/tags/v0.9.0-test",
3+
"ref_type": "tag",
4+
"repository": {
5+
"name": "gurbani-live",
6+
"full_name": "savitojs/gurbani-live"
7+
}
8+
}

‎.github/workflows/release.yml‎

Lines changed: 297 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,297 @@
1+
name: Release
2+
3+
on:
4+
push:
5+
tags:
6+
- "v*"
7+
workflow_dispatch:
8+
inputs:
9+
version:
10+
description: "Release version (e.g., v1.0.0)"
11+
required: true
12+
type: string
13+
14+
env:
15+
SCRIPT_NAME: gurbani-live
16+
17+
jobs:
18+
release:
19+
runs-on: ubuntu-latest
20+
needs: test
21+
permissions:
22+
contents: write
23+
24+
steps:
25+
- name: Checkout code
26+
uses: actions/checkout@v4
27+
28+
- name: Set version
29+
id: version
30+
run: |
31+
if [ "${{ github.event_name }}" == "workflow_dispatch" ]; then
32+
echo "VERSION=${{ github.event.inputs.version }}" >> $GITHUB_OUTPUT
33+
else
34+
echo "VERSION=${GITHUB_REF#refs/tags/}" >> $GITHUB_OUTPUT
35+
fi
36+
37+
- name: Update script with version
38+
run: |
39+
# Update version in script
40+
sed -i "s/VERSION=\".*\"/VERSION=\"${{ steps.version.outputs.VERSION }}\"/" gurbani-live
41+
42+
# Update SCRIPT_URL to point to the release script
43+
SCRIPT_URL="https://github.com/${{ github.repository }}/releases/download/${{ steps.version.outputs.VERSION }}/${{ env.SCRIPT_NAME }}"
44+
sed -i "s|SCRIPT_URL=\".*\"|SCRIPT_URL=\"${SCRIPT_URL}\"|" gurbani-live
45+
46+
- name: Prepare script for release
47+
run: |
48+
# Make script executable
49+
chmod +x ${{ env.SCRIPT_NAME }}
50+
51+
# Test the script
52+
./${{ env.SCRIPT_NAME }} --version
53+
54+
- name: Create release archive
55+
run: |
56+
# Create a tar.gz with the script and README
57+
tar -czf ${{ env.SCRIPT_NAME }}-${{ steps.version.outputs.VERSION }}.tar.gz ${{ env.SCRIPT_NAME }} README.md
58+
59+
# Create checksums
60+
sha256sum ${{ env.SCRIPT_NAME }} > ${{ env.SCRIPT_NAME }}.sha256
61+
sha256sum ${{ env.SCRIPT_NAME }}-${{ steps.version.outputs.VERSION }}.tar.gz > ${{ env.SCRIPT_NAME }}-${{ steps.version.outputs.VERSION }}.tar.gz.sha256
62+
63+
- name: Create Release
64+
uses: softprops/action-gh-release@v1
65+
with:
66+
tag_name: ${{ steps.version.outputs.VERSION }}
67+
name: Release ${{ steps.version.outputs.VERSION }}
68+
draft: false
69+
prerelease: false
70+
generate_release_notes: true
71+
files: |
72+
${{ env.SCRIPT_NAME }}
73+
${{ env.SCRIPT_NAME }}.sha256
74+
${{ env.SCRIPT_NAME }}-${{ steps.version.outputs.VERSION }}.tar.gz
75+
${{ env.SCRIPT_NAME }}-${{ steps.version.outputs.VERSION }}.tar.gz.sha256
76+
body: |
77+
## Gurbani Live - ${{ steps.version.outputs.VERSION }}
78+
79+
### Quick Installation (Recommended)
80+
81+
**Install latest stable release:**
82+
```bash
83+
curl -s https://github.com/${{ github.repository }}/releases/latest/download/gurbani-live | bash -s -- --install
84+
```
85+
86+
**Install this specific version:**
87+
```bash
88+
curl -s https://github.com/${{ github.repository }}/releases/download/${{ steps.version.outputs.VERSION }}/gurbani-live | bash -s -- --install
89+
```
90+
91+
**Install development version (main branch):**
92+
```bash
93+
curl -s https://raw.githubusercontent.com/${{ github.repository }}/main/gurbani-live | bash -s -- --install
94+
```
95+
96+
This will automatically download and install using the script's built-in interactive installer.
97+
98+
### Manual Installation
99+
100+
**Option 1: Download script directly**
101+
```bash
102+
# Download the script
103+
curl -L -o gurbani-live "https://github.com/${{ github.repository }}/releases/download/${{ steps.version.outputs.VERSION }}/gurbani-live"
104+
105+
# Make it executable
106+
chmod +x gurbani-live
107+
108+
# Move to PATH (optional)
109+
sudo mv gurbani-live /usr/local/bin/
110+
```
111+
112+
**Option 2: Download archive**
113+
```bash
114+
# Download and extract
115+
curl -L "https://github.com/${{ github.repository }}/releases/download/${{ steps.version.outputs.VERSION }}/gurbani-live-${{ steps.version.outputs.VERSION }}.tar.gz" | tar -xz
116+
117+
# Make executable and install
118+
chmod +x gurbani-live
119+
sudo mv gurbani-live /usr/local/bin/
120+
```
121+
122+
### Usage
123+
```bash
124+
gurbani-live --help
125+
```
126+
127+
### Verification
128+
Verify the download with checksums:
129+
```bash
130+
sha256sum -c gurbani-live.sha256
131+
```
132+
env:
133+
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
134+
135+
test:
136+
runs-on: ${{ matrix.os }}
137+
strategy:
138+
matrix:
139+
os: [ubuntu-latest, macos-latest]
140+
include:
141+
- os: ubuntu-latest
142+
platform: linux
143+
package_manager: apt-get
144+
install_cmd: "sudo apt-get update && sudo apt-get install -y"
145+
- os: macos-latest
146+
platform: macos
147+
package_manager: brew
148+
install_cmd: "brew install"
149+
150+
steps:
151+
- name: Checkout code
152+
uses: actions/checkout@v4
153+
154+
- name: Install dependencies on ${{ matrix.platform }}
155+
run: |
156+
echo "🧪 Testing on ${{ matrix.platform }}"
157+
158+
if [[ "${{ matrix.platform }}" == "linux" ]]; then
159+
# Install dependencies on Linux
160+
${{ matrix.install_cmd }} jq curl
161+
162+
# For VLC and fzf, try to install but don't fail if unavailable in CI
163+
${{ matrix.install_cmd }} vlc fzf || echo "VLC/fzf not available in CI environment"
164+
165+
elif [[ "${{ matrix.platform }}" == "macos" ]]; then
166+
# Install dependencies on macOS
167+
${{ matrix.install_cmd }} jq curl
168+
169+
# For VLC and fzf, try to install but don't fail if unavailable in CI
170+
${{ matrix.install_cmd }} fzf || echo "fzf not available in CI environment"
171+
# Note: VLC requires manual installation or cask on macOS CI
172+
echo "VLC would need to be installed via: brew install --cask vlc"
173+
fi
174+
175+
- name: Test script functionality on ${{ matrix.platform }}
176+
run: |
177+
echo "🔍 Testing script on ${{ matrix.platform }}..."
178+
179+
# Make script executable
180+
chmod +x ./gurbani-live
181+
182+
# Test basic functionality
183+
echo "Testing --version:"
184+
./gurbani-live --version
185+
186+
echo "Testing --help:"
187+
./gurbani-live --help | head -10
188+
189+
echo "Testing dependency checking:"
190+
# Test the check_dependencies function (this will show what's missing)
191+
timeout 10s ./gurbani-live --status || echo "Status check completed (expected timeout in CI)"
192+
193+
# Test network connectivity and GitHub API access
194+
echo "Testing GitHub API connectivity:"
195+
if command -v curl >/dev/null 2>&1; then
196+
timeout 10s curl -s "https://api.github.com/repos/${{ github.repository }}/releases/latest" | jq -r '.tag_name // "API accessible"' || echo "GitHub API test completed"
197+
fi
198+
199+
- name: Test macOS-specific compatibility
200+
if: matrix.platform == 'macos'
201+
run: |
202+
echo "🍎 Testing macOS-specific features..."
203+
204+
# Test readlink compatibility
205+
echo "Testing path resolution..."
206+
script_path="$(cd "$(dirname "./gurbani-live")" && pwd)/$(basename "./gurbani-live")"
207+
echo "Resolved path: $script_path"
208+
209+
# Test sed compatibility with a temporary file
210+
echo "Testing sed compatibility..."
211+
echo "test line" > /tmp/test_sed
212+
sed -i '' '/test/d' /tmp/test_sed && echo "macOS sed working correctly"
213+
rm -f /tmp/test_sed
214+
215+
# Test VLC.app detection
216+
echo "Testing VLC.app detection..."
217+
if [ -d "/Applications/VLC.app" ]; then
218+
echo "✅ VLC.app found"
219+
else
220+
echo "ℹ️ VLC.app not found (expected in CI)"
221+
fi
222+
223+
# Test VLC command availability (if installed via Homebrew)
224+
if command -v vlc >/dev/null 2>&1; then
225+
echo "✅ VLC command available"
226+
else
227+
echo "ℹ️ VLC command not available (expected in CI)"
228+
fi
229+
230+
- name: Test Linux-specific compatibility
231+
if: matrix.platform == 'linux'
232+
run: |
233+
echo "🐧 Testing Linux-specific features..."
234+
235+
# Test readlink -f
236+
echo "Testing readlink -f..."
237+
script_path="$(readlink -f "./gurbani-live")"
238+
echo "Resolved path: $script_path"
239+
240+
# Test sed without backup
241+
echo "Testing sed compatibility..."
242+
echo "test line" > /tmp/test_sed
243+
sed -i '/test/d' /tmp/test_sed && echo "Linux sed working correctly"
244+
rm -f /tmp/test_sed
245+
246+
# Test VLC command availability
247+
if command -v vlc >/dev/null 2>&1; then
248+
echo "✅ VLC command available"
249+
else
250+
echo "ℹ️ VLC command not available (expected in CI)"
251+
fi
252+
253+
- name: Test installation and update simulation
254+
run: |
255+
echo "🚀 Testing installation and update process..."
256+
257+
# Create a temporary directory to simulate installation
258+
mkdir -p /tmp/test_install
259+
export PATH="/tmp/test_install:$PATH"
260+
261+
# Test that script can find writable directories in PATH
262+
echo "Testing PATH detection..."
263+
./gurbani-live --help > /dev/null
264+
265+
# Test update functionality (dry run)
266+
echo "Testing update check..."
267+
timeout 10s ./gurbani-live --update --dry-run || echo "Update check completed (expected timeout in CI)"
268+
269+
# Test installation validation by verifying script can locate itself
270+
echo "Testing script self-location..."
271+
script_location="$(./gurbani-live --version 2>/dev/null | head -1 || echo "version check ok")"
272+
echo "Script responds to version check: $script_location"
273+
274+
echo "✅ Installation and update simulation completed successfully"
275+
276+
- name: Test one-liner installation method
277+
run: |
278+
echo "🌐 Testing one-liner installation method..."
279+
280+
# Test that the script can be executed via curl with --help flag
281+
echo "Testing script execution via curl (simulated)..."
282+
283+
# We can't actually install via curl in CI since we haven't released yet,
284+
# but we can test that the script handles the --install flag properly
285+
echo "Testing --install flag handling..."
286+
287+
# Test with various combinations that the install process expects
288+
timeout 5s ./gurbani-live --install --help 2>/dev/null || echo "Install help completed"
289+
290+
echo "✅ One-liner installation method tests completed"
291+
292+
- name: Summary
293+
run: |
294+
echo "🎉 All tests passed on ${{ matrix.platform }}!"
295+
echo "Platform: ${{ matrix.platform }}"
296+
echo "OS: ${{ matrix.os }}"
297+
echo "Package manager: ${{ matrix.package_manager }}"

‎CONTRIBUTING.md‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
1-
# Contributing to Gurbani Radio Fetch-n-Play
1+
# Contributing to Gurbani Live
22

3-
Thank you for considering contributing to Gurbani Radio Fetch-n-Play! We welcome all kinds of contributions, whether it's reporting bugs, suggesting features, or submitting code.
3+
Thank you for considering contributing to Gurbani Live! We welcome all kinds of contributions, whether it's reporting bugs, suggesting features, or submitting code.
44

55
## How to Contribute
66

0 commit comments

Comments
 (0)