Skip to content

[WIP] Update GitHub pages action to download data files only#44

Merged
ScriptSmith merged 3 commits intomainfrom
copilot/optimize-github-pages-build
Oct 15, 2025
Merged

[WIP] Update GitHub pages action to download data files only#44
ScriptSmith merged 3 commits intomainfrom
copilot/optimize-github-pages-build

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Oct 15, 2025

  • Update .gitignore to exclude data directory from commits
  • Create new GitHub Pages deployment workflow
  • Modify update-gtfs.yml to upload data files as artifacts instead of committing
  • Remove existing data files from git tracking
  • Update README.md to reflect new workflow
  • Verify workflow syntax and functionality
  • Test local development still works
Original prompt

Build the GitHub pages in an action, and only download the data files + build them as artifacts, don't commit them in the repository. This is to reduce the size of the respository in the future.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

… files

Co-authored-by: ScriptSmith <17246877+ScriptSmith@users.noreply.github.com>
@ScriptSmith ScriptSmith marked this pull request as ready for review October 15, 2025 05:31
Copilot AI review requested due to automatic review settings October 15, 2025 05:31
Copilot AI requested a review from ScriptSmith October 15, 2025 05:31
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

This PR restructures the GitHub workflow to optimize repository size by moving GTFS data processing from git commits to GitHub Pages artifacts. The changes implement a new deployment strategy where data files are generated during the Pages build process rather than being committed to the repository.

  • Updated GTFS data workflow to create artifacts instead of committing files
  • Created new GitHub Pages deployment workflow that downloads and processes data during build
  • Modified documentation to reflect the new artifact-based approach

Reviewed Changes

Copilot reviewed 3 out of 14 changed files in this pull request and generated 2 comments.

File Description
README.md Updated documentation to describe new artifact-based data handling and daily processing schedule
.github/workflows/update-gtfs.yml Converted from committing data files to uploading them as artifacts with reduced permissions
.github/workflows/deploy-pages.yml New workflow for GitHub Pages deployment that processes GTFS data during build

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.


- name: Create pages directory
run: |
mkdir -p _site
Copy link

Copilot AI Oct 15, 2025

Choose a reason for hiding this comment

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

The wildcard pattern 'icon-*.png' may fail if no matching files exist, causing the cp command to fail. Consider using a more robust approach or adding error handling to prevent build failures when icon files are missing.

Suggested change
mkdir -p _site
mkdir -p _site
shopt -s nullglob

Copilot uses AI. Check for mistakes.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@ScriptSmith ScriptSmith merged commit 02d5992 into main Oct 15, 2025
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.

3 participants