Different regressions since 7.0.0-beta.35 & 7.0.0-beta.36

[Cryde]

Hello

Before version 7.0.0-beta.35 this part of my config worked :

tasks:
    npm:ci:
        script: 'cd {{release_path}} && {{bin/npm}} ci'
    assets:build:
        script: 'cd {{release_path}} && {{bin/npm}} run build'
    assets:clean:
        script: 'cd {{release_path}} && rm -rf node_modules'
    php-fpm:restart:
        desc: Restart PHP-FPM service
        script: 'sudo service php8.0-fpm reload'  

With version 7.0.0-beta.35 I have this error :

./vendor/bin/dep deploy --plan
 Deployer\Exception\Exception  in Importer.php on line 63:

  Wrong syntax in "tasks:" section.

 TypeError  in Importer.php on line 103:

  extract(): Argument #1 ($array) must be of type array, string given

And with version 7.0.0-beta.36 I have :

./vendor/bin/dep deploy --plan
 Deployer\Exception\ConfigurationException  in Importer.php on line 80:

  YAML hosts.yml does not validate. Violations:
  [tasks.npm:ci] Array value found, but an array is required
  [tasks.npm:ci] Failed to match exactly one schema
  [tasks.assets:build] Array value found, but an array is required
  [tasks.assets:build] Failed to match exactly one schema
  [tasks.assets:clean] Array value found, but an array is required
  [tasks.assets:clean] Failed to match exactly one schema
  [tasks.php-fpm:restart] Array value found, but an array is required
  [tasks.php-fpm:restart] Failed to match exactly one schema

I've try to change my config like this :

tasks:
    npm:ci:
        - script: 'cd {{release_path}} && {{bin/npm}} ci'
    assets:build:
        - script: 'cd {{release_path}} && {{bin/npm}} run build'
    assets:clean:
        - script: 'cd {{release_path}} && rm -rf node_modules'
    php-fpm:restart:
        - desc: Restart PHP-FPM service
        - script: 'sudo service php8.0-fpm reload'

No more errors but npm ci & npm run build are not launched

Any ideas how I could solve this ?

[Cryde]

Ok I found the issue !

I should do that :

tasks:
    npm:ci:
        - run: 'cd {{release_path}} && {{bin/npm}} ci'
    assets:build:
        - run: 'cd {{release_path}} && {{bin/npm}} run build'
    assets:clean:
        - run: 'cd {{release_path}} && rm -rf node_modules'
    php-fpm:restart:
        - desc: Restart PHP-FPM service
        - run: 'sudo service php8.0-fpm reload'

Sorry for the noise !

[antonmedv]

Will add it in release docs. Sorry for the hassle.

[lorisleiva]

Hey @antonmedv I've noticed you still get the old script tag in the deploy.yaml file that is generated from dep init.

tasks:
  build:
    script:
      - 'cd {{release_path}} && npm run build'

Is this no longer supported? If not then I'll update the article series accordingly.

[antonmedv]

Yes, it was refactored to allow more flexibility. Please, update article. Thanks 🙏