[rush] git push --no-verify

[wavemotionio]

We are using rush change -v on the pre-push git hook to make sure that changes are logged prior to a push. Unfortunately, this interrupts publish from working correctly in the azure devops CI pipeline. The solution is to invoke git push --no-verify as demonstrated below. Is there a better way to do this?

Desired code:

- powershell: |
   git config --global user.name "Rushbot"
   git config --global user.email [email protected]
   node common/scripts/install-run-rush.js publish -a -b master --add-commit-details
  workingDirectory: '$(Build.Repository.LocalPath)'
  displayName: 'bump updated packages'
  failOnStderr: true
  condition: and(succeeded(), in(variables['Build.Reason'], 'IndividualCI', 'BatchedCI'))

Work around:

- powershell: |
   git config --global user.name "Rushbot"
   git config --global user.email [email protected]
   node common/scripts/install-run-rush.js version --bump
  workingDirectory: '$(Build.Repository.LocalPath)'
  displayName: 'bump updated packages'
  failOnStderr: true
  condition: and(succeeded(), in(variables['Build.Reason'], 'IndividualCI', 'BatchedCI'))

- powershell: |
   git checkout -b cirelease
  workingDirectory: '$(Build.Repository.LocalPath)'
  displayName: 'git checkout'
  failOnStderr: false
  condition: and(succeeded(), in(variables['Build.Reason'], 'IndividualCI', 'BatchedCI'))

- powershell: |
   git add -A
  workingDirectory: '$(Build.Repository.LocalPath)'
  displayName: 'git add'
  failOnStderr: false
  condition: and(succeeded(), in(variables['Build.Reason'], 'IndividualCI', 'BatchedCI'))

- powershell: |
   git reset HEAD file common/config/rush/.npmrc
   git commit -m "ci: version bump [skip ci]"
  workingDirectory: '$(Build.Repository.LocalPath)'
  displayName: 'git commit'
  failOnStderr: false
  continueOnError: true
  condition: and(succeeded(), in(variables['Build.Reason'], 'IndividualCI', 'BatchedCI'))

- powershell: |
   git push https://<pat>:[email protected]/project/_git/repo-name cirelease:master --no-verify
  workingDirectory: '$(Build.Repository.LocalPath)'
  displayName: 'git push'
  failOnStderr: false
  continueOnError: true
  condition: and(succeeded(), in(variables['Build.Reason'], 'IndividualCI', 'BatchedCI'))

[wavemotionio]

Possible solution: #1187

[octogonz]

This solution makes sense to me. We don't really want git hooks interfering with the automated CI operations.

It's tempting to enable rush install to disable the Git hooks entirely on a CI machine. But maybe the scalpel is wiser than the battle axe.

[wavemotionio]

Having an option to disable Git hooks entirely could be useful too but some folks might be using hooks for notifications and such. A more holistic approach might require an option that takes parameters.

[wavemotionio]

Closed this. Thanks for your help! Please reopen if the process is incomplete.