Commits on Sep 2, 2024

1. feat: allow conditional skip on success and fail comments (#874) …

   * feat: add `failCommentCondition` to `fail` script
   
   * feat: add `successCommentCondition` and `failCommentCondition` to `resolve-config`
   
   * feat: add `successCommentCondition` and `failCommentCondition` to `success` scipt
   
   * fix(build): lint
   
   * test: add `fail` case `Does not post comments if "failCommentCondition" is "false"`
   
   * test: add `fail` case for `Does not post comments on existing issues when "failCommentCondition" is "false"`
   
   * fix(build): lint
   
   * test(fail): add case for `Post new issue if none exists yet, but don't comment on existing issues when "failCommentCondition" is disallows it`
   
   * test(success): add case `Does not comment on issues/PR if "successCommentCondition" is "false"`
   
   * Update test/fail.test.js
   
   Co-authored-by: Jonas Schubert <jonas.schubert.projects@web.de>
   
   * test(success): add case for `Add comment and label to found issues/associatedPR using the "successCommentCondition"`
   
   * nits
   
   * test(success): add case for `Does not comment/label found associatedPR when "successCommentCondition" disables it`
   
   * test(success): improve case `Does not comment/label found associatedPR when "successCommentCondition" disables it`
   
   * doc: add documentation for `successCommentCondition` and `failCommentCondition`
   
   * Update lib/success.js
   
   Co-authored-by: Gregor Martynus <39992+gr2m@users.noreply.github.com>
   
   * refactor: modify `failTitle`, `failComment` and `successComment` false deprecation message
   
   * refator: implement early return in `fail` and `success` lifecyccle helper function
   
   * Update README.md
   
   Co-authored-by: Jonas Schubert <jonas.schubert.projects@web.de>
   
   * remove `failCommentCondition` example wrong description
   
   * build: fix lint
   
   * feat: add validators for `successCommentCondition` and `failCommentCondition`
   
   * feat: add `buildAssociatedPRs` to create pr object in form of issue object with pull_request property
   
   * doc: update README.md
   
   * build: fix lint
   
   * build: fix failing test `Add custom comment and labels`
   
   * feat: request more field for `associatedPRs` via graphql and improve `buildAssociatedPRs` response object with
   
   * test: modify integration tests
   
   * test: modify `success` unit tests
   
   * build: fix lint
   
   * build: fix failing tests
   
   * feat: add `__typename` to `issue.user` object
   
   * test: add new case `Does not comment/label associatedPR created by "Bots"`
   
   * test: modify `pull_request` mock value to `boolean`
   
   * chore(test): clean debug comments
   
   * feat: re-integrate `buildAssociatedPRs`
   
   * feat: re-introduced and modifed `loadSingleCommitAssociatedPRs`
   
   * refactor: introduce `parsedIssues` as returned value from `prs**.body` and `commits**.message`
   
   * feat: added `buildRelatedIssuesQuery` util graphql query builder
   
   * feat: implement computation for `responseRelatedIssues`
   
   * fix: correct `number` arg type in `buildRelatedIssuesQuery`
   
   * refactor: extract common field accross graphql queries to `baseFields`
   
   * refactor: transform `buildAssociatedPRs` to `buildIssuesOrPRsFromResponseNode` with ability to build both `PRs` and `Issues` object
   
   * feat: integrate `buildIssuesOrPRsFromResponseNode`
   
   * feat: implement `issueOrPR` for correctly addressing issues and pr in logs
   
   * feat: implement improved chunk operation helper `inChunks` and integrate in pr and issues fetch
   
   * build: fix lints
   
   * test: update `integrations` test
   
   * test: address PR and Issue naming in logs in `success`
   
   * refactor: why the `Promise.all()`? Removed it haha
   
   * feat: set default `type` param in `buildIssuesOrPRsFromResponseNode`
   
   * feat: address edge cases
   
   * test: fixed matchers in graphql request in `success` units
   
   * build: lint
   
   * docs: add ignore bots pr/issues example
   
   * fix: user issue `number` over `id`
   
   * test: improve case `'Does not comment/label associatedPR and relatedIssues created by "Bots"'`
   
   * doc: modify `buildIssuesOrPRsFromResponseNode` documentation
   
   ---------
   
   Co-authored-by: Jonas Schubert <jonas.schubert.projects@web.de>
   Co-authored-by: Gregor Martynus <39992+gr2m@users.noreply.github.com>

   

   3 people authored Sep 2, 2024
   1 Configuration menu

   • View commit details
   • Copy full SHA for e097d1c
   • Browse repository at this point

   Copy the full SHA

   e097d1c View commit details

   Browse the repository at this point in the history