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

Replace require-dev with webflo/drupal-core-require-dev #348

Merged
merged 2 commits into from
Jan 30, 2018

Conversation

webflo
Copy link
Member

@webflo webflo commented Jan 28, 2018

No description provided.

@webflo webflo mentioned this pull request Jan 28, 2018
@webflo webflo merged commit ffbdbf1 into drupal-composer:8.x Jan 30, 2018
@webflo webflo deleted the require-dev branch January 30, 2018 19:22
@wundo
Copy link

wundo commented Feb 1, 2018

@webflo why did you do that?

I could not find the reasoning for this change.

This change hides the require-dev dependencies inside an "empty" project, and makes it hard to override the dependencies on the fly. It broke mine (and probably others) continuous integration.

@webflo
Copy link
Member Author

webflo commented Feb 1, 2018

@wundo I am sorry about that, but this project its a starterkit. I think you should not depend on it directly. Use it as a base, make it your own.

Back to this specific change. The development dependencies got outdated over time. It was my goal to provide it as and standalone packages. I hope this makes it easier to update your project later on.

hawkeyetwolf pushed a commit to kalamuna/drupal-project that referenced this pull request Mar 9, 2018
…m-project

* upstream/8.x:
  Improve update instructions (drupal-composer#366)
  Run tests with 8.6.x
  Update to Drupal 8.5
  Add platform version to FAQ to help user specifying PHP version, fixes drupal-composer#359. (drupal-composer#360)
  Update /drush dir for Drush9 (drupal-composer#350)
  Revert "Update /drush dir for Drush9"
  Update /drush dir for Drush9
  Replace require-dev with webflo/drupal-core-require-dev (drupal-composer#348)
  Bump drush version
  Replace GPL-2.0+ by GPL-2.0-or-later
  Run tests with 8.5.x (drupal-composer#329)
  Update README.md (drupal-composer#334)
  Update composer dependencies (drupal-composer#325)
  Declare compatibility with versions of Drush and CSS selector that are compatible with Drupal 8.4 and earlier.
  Added composer validate to CI.
  Update .travis.yml
  Update .travis.yml
  Add support for Drush 9 and Drupal 8.4
daften pushed a commit to district09/drupal8_project_template that referenced this pull request Aug 9, 2018
* upstream/8.x: (22 commits)
  Remove unused use statement. (drupal-composer#406)
  Fix typo in README.md
  Remove "drupal-scaffold" from scripts (drupal-composer#275)
  Copy the editor config and gitattributes from drupal to the root. (drupal-composer#404)
  Fix tests on Travis CI (drupal-composer#405)
  Update drupal-composer/drupal-scaffold
  Fix installer path for drush commandfiles.
  Bump minimal version to 8.5.3
  Bump minimal version
  Updated argument name
  Remove extra whitespace (drupal-composer#371)
  Honor .env files and suggest usage (drupal-composer#351)
  Improve update instructions (drupal-composer#366)
  Run tests with 8.6.x
  Update to Drupal 8.5
  Add platform version to FAQ to help user specifying PHP version, fixes drupal-composer#359. (drupal-composer#360)
  Update /drush dir for Drush9 (drupal-composer#350)
  Revert "Update /drush dir for Drush9"
  Update /drush dir for Drush9
  Replace require-dev with webflo/drupal-core-require-dev (drupal-composer#348)
  ...

# Conflicts:
#	.gitignore
#	composer.json
#	drush/policy.drush.inc
shrop pushed a commit to shrop/drupal-project that referenced this pull request Apr 11, 2020
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