[GitHub] [couchdb-helm] colearendt commented on pull request #80: Add CI via GitHub Actions

[GitBox <gi...@apache.org>]

colearendt commented on PR #80:
URL: https://github.com/apache/couchdb-helm/pull/80#issuecomment-1158654891

   TIL about `.asf.yaml`! Nice!
   
   I have made the discussed changes:
   
   - run `lint`-only changes on branches (still running `all` on main because there are never any changes otherwise. Testing/linting on main still seems desirable). It is pretty simple to revert this at some point, and is low cost anyways. One benefit is consistency with the `install` checks
   - Add `--upgrade` flag to `ct install` tests
   
   One last idea:
   - wanted to make sure that `push to main` and `pull_request` is the right definition for triggers. i.e. pushing to a random branch will not trigger CI unless a PR is created. IIRC one of the problems of just using "push to any branch" was that it did not trigger CI for remote PRs, and having both "all branches" and "PR"s would trigger duplicate CI runs on PRs.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@couchdb.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org