-
-
Notifications
You must be signed in to change notification settings - Fork 0
Updated Vortex to 'ca0ca82'. #317
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
Changes from all commits
4dac17d
743550e
28f39c1
6dceb2a
a7af5bb
fc344be
71ac027
f142527
d0592a4
8e19086
aacdb13
0618f99
42a63da
6257f12
6fe51fc
95d15d6
9587535
fb5bb80
c644090
c57f2ed
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -93,9 +93,10 @@ RUN mkdir -p -m 2775 "/app/${WEBROOT}/${DRUPAL_PUBLIC_FILES}" "/app/${WEBROOT}/$ | |
|
|
||
| RUN if [ "${VORTEX_FRONTEND_BUILD_SKIP}" != "1" ]; then \ | ||
| theme_path="/app/${WEBROOT}/themes/custom/${DRUPAL_THEME}"; \ | ||
| npm --prefix "${theme_path}" ci --no-progress && \ | ||
| npm --prefix "${theme_path}" run build && \ | ||
| npm cache clean --force; \ | ||
| export npm_config_cache=/tmp/npm-cache; \ | ||
|
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Vortex The theme build now points npm's cache at
|
||
| npm --prefix="${theme_path}" ci --no-progress --no-audit --no-fund && \ | ||
| npm --prefix="${theme_path}" run build && \ | ||
| rm -rf /tmp/npm-cache; \ | ||
| fi | ||
|
|
||
| WORKDIR /app | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,5 +1,8 @@ | ||
| # check=skip=SecretsUsedInArgOrEnv | ||
|
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Vortex Adds No image tag or base image change here, so nothing about what the container runs is affected. |
||
| # Database container. | ||
| # | ||
| # The check skipped above is BuildKit's twin of DL3064, ignored inline below. | ||
| # | ||
| # @see https://hub.docker.com/r/uselagoon/mysql-8.4/tags | ||
| # @see https://github.com/uselagoon/lagoon-images/tree/main/images/mysql | ||
| # | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -23,7 +23,7 @@ indent_size = 4 | |
| [*.xml] | ||
| indent_size = 4 | ||
|
|
||
| [.eslintrc.json] | ||
| [eslint.config.mjs] | ||
|
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Vortex Follows the ESLint 9 migration: the two-space indent rule was keyed to |
||
| indent_size = 2 | ||
|
|
||
| [.prettierrc.json] | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -97,7 +97,7 @@ DRUPAL_CLAMAV_MODE=daemon | |
| # or fresh install from profile), running updates, appying configuration | ||
| # changes, clearing caches and performing other tasks that prepare the site for | ||
| # use. | ||
| # @see https://www.vortextemplate.com/docs/drupal/provision | ||
| # @see https://www.vortextemplate.com/docs/development/provision | ||
|
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Vortex Documentation links only. Vortex restructured its docs into a single Worth noting for what is not here: adopting Reroute Email in this PR did not require new |
||
|
|
||
| # Set to 'profile' to install a site from profile instead of the database dump. | ||
| VORTEX_PROVISION_TYPE=database | ||
|
|
@@ -126,13 +126,13 @@ VORTEX_PROVISION_VERIFY_CONFIG_UNCHANGED_AFTER_UPDATE=0 | |
| # | ||
| # Database sanitization is enabled by default in all non-production | ||
| # environments and is always skipped in the production environment. | ||
| # @see https://www.vortextemplate.com/docs/drupal/provision#database-sanitization | ||
| # @see https://www.vortextemplate.com/docs/development/provision#database-sanitization | ||
| VORTEX_PROVISION_SANITIZE_DB_SKIP=1 | ||
|
|
||
| # Sanitization email pattern. | ||
| # | ||
| # Applied if database sanitization is enabled. | ||
| # @see https://www.vortextemplate.com/docs/drupal/provision#database-sanitization | ||
| # @see https://www.vortextemplate.com/docs/development/provision#database-sanitization | ||
| VORTEX_PROVISION_SANITIZE_DB_EMAIL=user_%uid@drevops.com | ||
|
|
||
| # Put the site into a maintenance mode during site provisioning. | ||
|
|
||
This file was deleted.
This file was deleted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Vortex
ca0ca82: drevops/vortex#3046Every front-end command moves from Yarn to npm.
ahoy feinow runsnpm ciat the root andnpm --prefix=... cifor the subtheme, andfe/fed/few/lint-fe/lint-fix-fecallnpm runinstead ofyarn run.The practical effect: a root
node_modulesinstalled by Yarn should be removed once after checking this branch out, ornpm ciwill be working against a tree it did not create. The subtheme is unaffected because it already used npm.The template also renames the Jest
test-jscommand in this commit; it is absent here because this project does not have Jest enabled.