Fix update/merge edge cases

Fixed

• fixed updating PR accidentally removing it from the merge queue (#148)
• fixed possible race condition when dont_wait_on_status_checks was configured that could accidentally remove a PR from the merge queue (#149)

ignore pending status checks and fix review evaluations

Added

• display requested reviewer names in status messages (#130)
• add warning that forks cannot be updated when PR is from fork (#135)
• add nicer error message for unknown block reason. The previous message erroneously indicated there was a problem with Kodiak (#139)
• add configuration to ignore select pending status checks. This is useful to prevent waiting indefinitely for the WIP GitHub App Check Run to complete (#141)

Fixed

• ensure user has write permissions when counting their reviews towards mergeability. We previously checked the wrong field for this information. (#134)

Better status messages and reliability

Added

• configuration for redis connection pool size (#57)
• merge.optimistic_updates configuration to prioritize updates over status checks (#64)
• Internal rate limiting of API calls to Github (#70)
• Improve status check messages when PR is being merged (#72)
• Add queue position information to status messages (#74)
• Display more information in status checks (#77, #112)
  • has blacklist labels reports the blacklist labels
  • invalid merge method displays the configured merge method and the valid merge methods
  • blocking review shows first blocking user
  • missing required review count shows review count and required review count
  • failing required status checks displays the failing status checks
  • waiting on status checks now reports the status checks we are waiting for
• Display status messaging for missing branch protection (#78)
• Add merge.message.strip_html_comments configuration option to strip comments from (#80)
  • This is useful for stripping HTML comments created by PR templates when the markdown body_type is used.
• Add require_automerge_label to configure requirement of automerge_label for working on PR (#82)
  • This overrides and disables notify_on_conflict (#86)
• Add status message warning for unsupported requiresCommitSignatures branch protection configuration (#90, #91)
  • This is a limitation of the Github API as Kodiak is not able to created signed commits when merging a PR.
• Add status message reporting of update branch failures (#94)
• Add "Known issues" section to README (#105, #114)
• Add 'empty' configuration option for merge.message.body to truncate PR body on merge (#111)
• Display configuration parsing errors with details page when kodiak cannot parse a configuration file (#116, #125)

Fixed

• Support PRs in draft state (#68)
• Fix bug where require_automerge_label would trigger kodiak to make an infinite loop of comments (#86)
• Fix poor status message templating on update branch failure case (#110)

v0.4.0

container: cdignam/kodiak:v0.4.0

Added

• Add support for include_pr_number configuration. Enabling include_pr_number with a non-default merge message option will append the pr number to the commit message, like the Github UI.
• Make merge body style (plain text, markdown, or HTML) configurable via merge.message.body_type.
• Add automatic deletion of branches on merge configurable via merge.delete_branch_on_merge.
• Add support for running multiple kodiak instances on the same repo via the app_id configuration option.
• Add redis-based persistence. Redis >=5 is now required.
• Add merge.blacklist_title_regex configuration to block merging PRs that match configured regex.
• Display kodiak status information in Github CheckRun.
• Add docs for testing kodiak locally.
• Add automerge label removal and PR comment when a merge conflict occurs. This is configurable via merge.notify_on_conflict.

Changed

• move block_on_reviews_requested to merge.block_on_reviews_requested.
• replace merge.whitelist array with singular merge.automerge_label.
• rename merge.blacklist to merge.blacklist_labels.

v0.3.0

container: cdignam/kodiak:v0.3.0

Added

• Add support for configuring merge messages. Current options are default Github style or using pull request title and body.

Fixed

• Fix handling of CHANGE_REQUESTED reviews. We weren't nullifying CHANGE_REQUESTED reviews after the user placed another review.

v0.2.1

container: cdignam/kodiak:v0.2.1

Fixed

• Fix incorrect calls to sentry client. send_message => capture_message
• Fix missing handling for Github CheckRuns.

v0.2.0

container: cdignam/kodiak:v0.2.0

CHANGELOG

Added

• Add block_on_reviews_requested configuration to block merging if there are unanswered review requests.

Changed

• Update mergeability evaluation to trigger branch update after all other mergeability tests are verified.

v0.1.0

Initial release

container: cdignam/kodiak:v0.1.0