Skip to content

Conversation

@davidsbailey
Copy link
Member

@davidsbailey davidsbailey commented Dec 17, 2025

The goal of this PR is to move any files we need out of pegasus/sites.v3, for two reasons:

  1. make it easier to delete pegasus/sites and pegasus/sites.v3 entirely when we're ready to remove pegasus content
  2. unblock us from omitting pegasus content more easily via sparse checkout and the no-pegasus-content file in the interim. specifically, it's hard to keep git LFS from pulling those pegasus LFS files down into the .git directory when there is a complicated pattern in no-pegasus-content

Done in this PR:

  • move config.json and countries.json files out of sites.v3 and into pegasus/config/sites
  • remove entries from no-pegasus-content which have already been removed from the repo

Testing story

  • pegasus tests passing after moving config json files (pegasus tests depend on these files)
  • passing drone run with [skip pegasus content] and all unit tests

Follow-up work

address remaining TODOs in no-pegasus-content file


# used by pegasus-specific dcdo_mocking.feature UI test
# TODO: remove once we no longer use DCDO flags in pegasus
pegasus/sites.v3/code.org/public/test_dcdo.haml
Copy link
Member Author

Choose a reason for hiding this comment

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


# placeholder image used by pegasus hoc and poste routes
# TODO: use existing dashboard/app/assets/images/1x1.png instead
pegasus/sites.v3/code.org/public/images/1x1.png
Copy link
Member Author

Choose a reason for hiding this comment

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

@davidsbailey davidsbailey marked this pull request as ready for review December 17, 2025 03:16
Copy link
Member

@artem-vavilov artem-vavilov left a comment

Choose a reason for hiding this comment

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

Good job 🙌🏻

@davidsbailey davidsbailey merged commit 1e45da2 into staging Dec 17, 2025
6 checks passed
@davidsbailey davidsbailey deleted the move-needed-pegasus-content-files branch December 17, 2025 18:12
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.

2 participants