Skip to content

Add some updates#14

Merged
aculix merged 5 commits intoaculix:mainfrom
nandyalu:main
Jul 11, 2025
Merged

Add some updates#14
aculix merged 5 commits intoaculix:mainfrom
nandyalu:main

Conversation

@nandyalu
Copy link
Copy Markdown
Contributor

Here are the changes:

  • Added support for opening BitPlay with torrent query parameter. Updated README.md with an example.
  • New GitHub Action workflow to run on releases with a Sem Ver tag (vA.B.C). Uses GitHub Cache for build artifacts. Fixes Add semantic versioning #2
  • Also added an ARG in Dockerfile which will be set during the build process to the Sem Ver from the tag.
  • Use local assets instead of CDN. Fixes Remove video player cdn #1

Hope this helps!

Copilot AI review requested due to automatic review settings July 10, 2025 22:18
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Adds local asset support, torrent-query prefill functionality, and automates Docker image publishing on semver-tagged releases.

  • Replaced CDN references with local assets (video-js CSS/JS) and added script to read torrent query parameter into the magnet input.
  • Documented the new torrent query parameter usage in README.md.
  • Introduced an APP_VERSION build arg in the Dockerfile and added a GitHub Actions workflow for building, caching, and signing Docker images on semver tags.

Reviewed Changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
client/index.html Replaced video.js CDN links with local assets and added DOMContentLoaded handler for torrent query parameter
client/assets/video-js.min.css Added local copy of video-js CSS v8.10.0
README.md Expanded usage instructions with example for torrent query parameter
Dockerfile Added ARG APP_VERSION to capture build-time semver tag
.github/workflows/docker-publish-release.yml New workflow: caches build, builds/pushes signed Docker images on semver-tagged pushes
Comments suppressed due to low confidence (1)

.github/workflows/docker-publish-release.yml:53

  • The cache key references package.json which doesn’t exist in this repo, so caching will always miss. Consider hashing go.mod or just the Dockerfile instead.
          key: ${{ runner.os }}-docker-${{ hashFiles('package.json', 'Dockerfile') }}

nandyalu and others added 2 commits July 10, 2025 17:21
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@aculix aculix merged commit ce2c04e into aculix:main Jul 11, 2025
1 check passed
@nandyalu
Copy link
Copy Markdown
Contributor Author

@aculix now create a new release with a SemVer tag like v1.1.0.

You will be asked to select/create a tag while creating a release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add semantic versioning Remove video player cdn

3 participants