Skip to content
Snippets Groups Projects

Add composer.json for easier inclusion in projects

Merged bgm requested to merge bgm/firewall:addComposer into master
1 unresolved thread

Makes it easier to handle extension updates in composer-managed projects.

(I'm messing around with composer so that it's easier for our shop to update our various builds, track versions, etc)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
    • @sluc23 thoughts?

    • looks good to me :thumbsup:
      There's another property that is useful in composer.json, which is type, but there's not yet a clear general convention about which value to use here.. we use internally "type": "civicrm-extension" (as here ).. it helps to place extensions automatically in an specific folder, for example if defined in the main composer.json as

      {
          . . .
          "repositories": [
              . . .
          ],
          "require": {
              . . .
          },
          "extra": {
              . . .
              "installer-paths": {
                  . . .
                  "web/modules/contrib/{$name}": [
                      "type:drupal-module"
                  ],
                  "web/themes/contrib/{$name}": [
                      "type:drupal-theme"
                  ],
                  "vendor/civicrm/{$name}": [
                      "type:civicrm-extension"
                  ]
              },
          },
          . . .
      }
      

      but this is a discussion we can have in a different issue

      Edited by sluc23
    • Author Owner

      @sluc23 Good point - I added the type property.

    • Please register or sign in to reply
  • bgm added 1 commit

    added 1 commit

    • d641fe71 - Add composer.json for easier inclusion in projects

    Compare with previous version

  • mattwire mentioned in commit 5dada8bc

    mentioned in commit 5dada8bc

  • merged

Please register or sign in to reply
Loading