docs: Revert "docs: add download page for app branch binaries" - #3444
Merged
Merged
Conversation
This reverts commit 33e7db4.
Contributor
|
🧙 Sourcery has finished reviewing your pull request! Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Contributor
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- Since this reverts a previously added download-page generation due to workflow failure, consider adding a brief comment near the affected workflow section explaining why it was removed to prevent future reintroductions without addressing the underlying CI issue.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Since this reverts a previously added download-page generation due to workflow failure, consider adding a brief comment near the affected workflow section explaining why it was removed to prevent future reintroductions without addressing the underlying CI issue.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Contributor
There was a problem hiding this comment.
Pull request overview
This PR reverts the previously introduced “app-branch binaries download page” feature, restoring the prior documentation-driven download flow and removing the CI-generated HTML index that was causing post-merge workflow failures.
Changes:
- Update the root README to point users to the official PSLab Application documentation for downloads/install instructions.
- Remove the dedicated
docs/downloads.htmldevelopment-builds download page. - Delete the GitHub Actions workflow step that generated and published an
index.htmldownload page onto theappbranch.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
README.md |
Removes the direct “development builds” download-page link and directs users to the canonical documentation. |
docs/downloads.html |
Deletes the standalone downloads HTML page that listed app-branch artifacts. |
.github/workflows/push-event.yml |
Removes the workflow logic that generated an HTML download index during artifact publication to the app branch. |
marcnause
enabled auto-merge (rebase)
July 26, 2026 21:06
Contributor
Build StatusBuild successful. APKs to test: https://github.com/fossasia/pslab-app/actions/runs/30220194591/artifacts/8637102009. Screenshots |
CloudyPadmal
approved these changes
Jul 27, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.





















Reverts #3435
Reason: https://github.com/fossasia/pslab-app/actions/runs/30219313633 fails after merging
Summary by Sourcery
Revert automatic generation and publication of the app-branch binary download page, returning to the previous documentation-driven download flow.
Documentation:
Chores: