-
Notifications
You must be signed in to change notification settings - Fork 140
feat: allow conditional skip on success and fail comments #874
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
79 commits
Select commit
Hold shift + click to select a range
304d023
feat: add `failCommentCondition` to `fail` script
babblebey 120d391
feat: add `successCommentCondition` and `failCommentCondition` to `re…
babblebey ec8f4d6
feat: add `successCommentCondition` and `failCommentCondition` to `su…
babblebey 508e066
fix(build): lint
babblebey f319896
test: add `fail` case `Does not post comments if "failCommentConditio…
babblebey 5983b75
test: add `fail` case for `Does not post comments on existing issues …
babblebey da630b1
Merge branch 'master' into feat/conditional-success-fail-comment-skip
babblebey a71ee8a
Merge branch 'master' into feat/conditional-success-fail-comment-skip
babblebey 8231ec0
Merge branch 'master' into feat/conditional-success-fail-comment-skip
babblebey f516292
fix(build): lint
babblebey 460d475
test(fail): add case for `Post new issue if none exists yet, but don'…
babblebey 7d2b4de
test(success): add case `Does not comment on issues/PR if "successCom…
babblebey e55733b
Update test/fail.test.js
babblebey 2156d92
test(success): add case for `Add comment and label to found issues/as…
babblebey 6f46d09
nits
babblebey e32cc69
test(success): add case for `Does not comment/label found associatedP…
babblebey 30b6f38
Merge branch 'feat/conditional-success-fail-comment-skip' of https://…
babblebey 5d9e5d5
Merge branch 'master' into feat/conditional-success-fail-comment-skip
babblebey 81c9ff0
test(success): improve case `Does not comment/label found associatedP…
babblebey a82c2b8
doc: add documentation for `successCommentCondition` and `failComment…
babblebey 219386c
Update lib/success.js
babblebey 34ae9a9
Merge branch 'master' into feat/conditional-success-fail-comment-skip
babblebey 1cbf8c2
refactor: modify `failTitle`, `failComment` and `successComment` fals…
babblebey 0d3014b
refator: implement early return in `fail` and `success` lifecyccle he…
babblebey ad0a03f
Update README.md
babblebey 1faae48
remove `failCommentCondition` example wrong description
babblebey 924ca55
build: fix lint
babblebey 66c9f28
Merge branch 'master' into feat/conditional-success-fail-comment-skip
babblebey a556072
Merge branch 'master' into feat/conditional-success-fail-comment-skip
babblebey 7328c8c
Merge branch 'master' into feat/conditional-success-fail-comment-skip
babblebey cfc61eb
feat: add validators for `successCommentCondition` and `failCommentCo…
babblebey 9591bd0
feat: add `buildAssociatedPRs` to create pr object in form of issue o…
babblebey 1cbb41a
doc: update README.md
babblebey 8cfa350
build: fix lint
babblebey 6ee6c24
build: fix failing test `Add custom comment and labels`
babblebey f250e71
feat: request more field for `associatedPRs` via graphql and improve …
babblebey 60a73d6
test: modify integration tests
babblebey 4b689fe
test: modify `success` unit tests
babblebey ee733fb
Merge branch 'master' into feat/conditional-success-fail-comment-skip
babblebey a798fc2
Merge branch 'master' into feat/conditional-success-fail-comment-skip
babblebey 09cfdac
Merge branch 'master' into feat/conditional-success-fail-comment-skip
babblebey a9af537
Merge branch 'master' into feat/conditional-success-fail-comment-skip
babblebey 627b0d8
build: fix lint
babblebey 863b4a6
build: fix failing tests
babblebey 2d826fa
feat: add `__typename` to `issue.user` object
babblebey 990ed77
test: add new case `Does not comment/label associatedPR created by "B…
babblebey 9e19862
test: modify `pull_request` mock value to `boolean`
babblebey a052c49
chore(test): clean debug comments
babblebey 260bdd6
Merge branch 'master' into feat/conditional-success-fail-comment-skip
babblebey 54dd036
feat: re-integrate `buildAssociatedPRs`
babblebey a6c8442
feat: re-introduced and modifed `loadSingleCommitAssociatedPRs`
babblebey e0e7ff1
refactor: introduce `parsedIssues` as returned value from `prs**.body…
babblebey c068bb3
feat: added `buildRelatedIssuesQuery` util graphql query builder
babblebey 63a4bc1
feat: implement computation for `responseRelatedIssues`
babblebey 229d4a8
fix: correct `number` arg type in `buildRelatedIssuesQuery`
babblebey 43ca24f
refactor: extract common field accross graphql queries to `baseFields`
babblebey c4c561d
refactor: transform `buildAssociatedPRs` to `buildIssuesOrPRsFromResp…
babblebey 88a7118
feat: integrate `buildIssuesOrPRsFromResponseNode`
babblebey 1bc293c
feat: implement `issueOrPR` for correctly addressing issues and pr in…
babblebey e95a700
feat: implement improved chunk operation helper `inChunks` and integr…
babblebey 2572def
build: fix lints
babblebey 201a1bb
Merge branch 'master' into feat/conditional-success-fail-comment-skip
babblebey d41a1df
test: update `integrations` test
babblebey 6473eb9
test: address PR and Issue naming in logs in `success`
babblebey 94f3975
refactor: why the `Promise.all()`? Removed it haha
babblebey e29b312
feat: set default `type` param in `buildIssuesOrPRsFromResponseNode`
babblebey 474a28a
feat: address edge cases
babblebey cbd0128
Merge branch 'master' into feat/conditional-success-fail-comment-skip
babblebey 1ef47af
Merge branch 'feat/conditional-success-fail-comment-skip' of https://…
babblebey 34795a5
Merge branch 'master' into feat/conditional-success-fail-comment-skip
babblebey 8ee3679
test: fixed matchers in graphql request in `success` units
babblebey f617843
build: lint
babblebey e336619
Merge branch 'feat/conditional-success-fail-comment-skip' of https://…
babblebey 15c3089
docs: add ignore bots pr/issues example
babblebey f91e8e9
Merge branch 'master' into feat/conditional-success-fail-comment-skip
babblebey ff17066
fix: user issue `number` over `id`
babblebey 601303d
test: improve case `'Does not comment/label associatedPR and relatedI…
babblebey 367047d
doc: modify `buildIssuesOrPRsFromResponseNode` documentation
babblebey 0d3cd0f
Merge branch 'master' into feat/conditional-success-fail-comment-skip
babblebey File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.