Skip to content
This repository has been archived by the owner on Feb 13, 2023. It is now read-only.

Update README.md - installing Drupal 8 #2083

Merged
merged 2 commits into from
Sep 30, 2020
Merged

Conversation

ijf8090
Copy link
Contributor

@ijf8090 ijf8090 commented Sep 10, 2020

First ever pull request - please be gentle:)

I got very confused trying to override the default Drupal 9 installation. I'm proposing an addition to the README to add some clarification. It's far from perfect, there are a couple of additions I would like to make but don't know how to

  1. I don't understand the relationship between drupal_major_version and drupal_composer_project_package. drupal�_major_version seems to have no affect on the default install, but somewhere on the web (can't find it now) it is the recommended solution. Perhaps just an update to the default.config.yml to link the two variables together would help?

2, I would like to add a pointer to the available versions e.g. https://www.drupal.org/project/drupal/releases or "For a specific version, use drupal/recommended-project:8.8.0. For all available versions, use the shell with composer show drupal/recommended-project --all."

README.md Outdated Show resolved Hide resolved
@geerlingguy geerlingguy merged commit 2a0b13e into geerlingguy:master Sep 30, 2020
@geerlingguy
Copy link
Owner

Looks great, was just missing one backtick, which I added pre-merge. Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants