acquia / cli

Command-line interface for Acquia Cloud Platform products
https://docs.acquia.com/acquia-cli/
GNU General Public License v2.0
42 stars 45 forks source link

GL-2582 : Remove schedule pipelines for node projects #1759

Closed akashkska closed 3 weeks ago

akashkska commented 3 weeks ago

As of now schedule pipelines are configured for both node projects and drupal projects. But those are only required for Drupal Projects. Hence removing it from node projects.

Manual tests:

  1. With the Node project selected in the cs:wizard command, the resultant Node project in Gitlab should not have a pre-configured scheduled pipeline
  2. Regression test: With this update, the Drupal selection flow should not be impacted (i.e. creation, variables etc. along with a configured scheduled pipeline should be validated)
codecov[bot] commented 3 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 91.75%. Comparing base (5a4966d) to head (6f4d898).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1759 +/- ## ========================================= Coverage 91.75% 91.75% Complexity 1818 1818 ========================================= Files 121 121 Lines 6524 6524 ========================================= Hits 5986 5986 Misses 538 538 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

github-actions[bot] commented 3 weeks ago

Try the dev build for this PR: https://acquia-cli.s3.amazonaws.com/build/pr/1759/acli.phar

curl -OL https://acquia-cli.s3.amazonaws.com/build/pr/1759/acli.phar
chmod +x acli.phar