Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhance/846b tb cleanup phase1dot5 #889

Open
wants to merge 111 commits into
base: master
Choose a base branch
from

Conversation

gilzow
Copy link
Contributor

@gilzow gilzow commented Jul 27, 2023

Closes #846 and partially completes #825
This is a big one. For 27 templates designated as starters

  • it removes all non-platform.sh configuration files from their areas in template builder. For those removed files, management and tracking of those files are moved to their individual repositories.
  • platform.sh configuration files have been resynced with the versions in their individual templates as they have drifted over time
  • platform.sh configuration files have had their internal properties re-ordered so that the list of properties matches the order in the documentation
  • platform.sh configuration files have had internal documentation added to each property that includes a brief explanation and a link to the relevant docs for more/complete information
  • for the above templates that have a remote source (e.g. Drupal10), new functionality has been added as a mixin to address the renaming of the upstream README (to README_UPSTREAM) so it doesn't clobber our README that we need in the templates' repository
  • Additional clean-up was made as discovered (correcting cron syntax, updating template builder commands for a template where things have changed, updating runtimes, etc)
  • Pushes out new github workflows to all the above templates that enables auto-updating via source operations

gilzow and others added 30 commits May 30, 2023 13:24
 .environment
 .gitignore
 .lando.upstream
 README
 wp-cli.yml
 wp-config.php
…o copy wp-config.php and wp-cli.yml from files into the wordpress directory since they are now there via the clone from the template repo
.blackfire.yml
.editorconfig
.environment
.gitattributes
.gitignore
.lando.upstream.yml
header.svg
php.ini
READM.md

Additionally, removes the following:
.ddev/*
.github/*
config/sync
drush/*
web/sites/default/settings.php
web/sites/default/settings.platformsh.php
.editorconfig
.environment
.gitignore
header.svg
READM.md

Additionally, removes the following:
config/sync
drush/*
web/sites/default/settings.php
web/sites/default/settings.platformsh.php
.editorconfig
.environment
.gitignore
.lando.upstream.yaml
header.svg
READM.md
psh-subsite-add.php

Additionally, removes the following:
config/sync/*
drush/*
web/sites/* which includes settings.php, services.yaml, and drushrc.php for default, main, first, and second directories
web/sites/default.settings.php
web/sites/settings.platformsh.php
web/sites/default.services.yaml
.blackfire.yaml
.editorconfig
.environment
.gitattributes
.gitignore
.lando.upstream.yaml
govcms-logo.png
govcms-logo.svg
header.svg
php.ini
platformsh_deploy_drush.sh
READM.md
psh-subsite-add.php

Additionally, removes the following:
.ddev/*
.github/*
config/sync/*
drush/*
docroot/sites/default/*
patches/*
…php version to 8.2 between version 3.0.0 and 3.1.0 : https://github.com/lcobucci/clock/releases/tag/3.1.0

For now locking to 3.0.0 until we upgrade this project to php 8.2
change `More info` to `More information`
…ormsh/template-builder into enhance/846b-tb-cleanup-phase1dot5
Updates Pipfile to match what is in the current instance of the template repo
gilzow added 24 commits July 24, 2023 09:26
Adds env var to config to disable telemetry
syncs platformsh config files for magento from template repository
… anything beyond what is done in BaseProject

Removes the import of the magento class in the baseproject file
…for the following templates:

beego
directus
django3
django4
echo
express
fastapi
flask
gatsby
gin
koa
laravel
nextjs
pyramid
strapi4
wagtail
@gilzow gilzow requested a review from chadwcarlson July 27, 2023 20:05
@gilzow gilzow self-assigned this Jul 27, 2023
readMeUpstreamFile = "_UPSTREAM"
readMePSHFile = "_PSH"

def rename_psh_readme(self):
Copy link
Contributor

Choose a reason for hiding this comment

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

Mixin!

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.

[Enhancement] Phase 1 of template clean-up
3 participants