Merge pull request danger#1340 from rogerluan/fix-draft-status

Fix `pr_draft?` GitHub Method
manicmaniac · Dec 17, 2021 · 4300005 · 4300005
2 parents 4e1756d + 834fd88
commit 4300005
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -10,6 +10,7 @@
 
 ## master
 <!-- Your comment below here -->
+* Fix issue in GitHub `pr_draft?` method which was returning false for draft Pull Requests - [@rogerluan](https://github.com/rogerluan)
 <!-- Your comment above here -->
 
 ## 8.4.2

diff --git a/lib/danger/danger_core/plugins/dangerfile_github_plugin.rb b/lib/danger/danger_core/plugins/dangerfile_github_plugin.rb
@@ -148,7 +148,7 @@ def pr_labels
  # @return [Boolean]
  #
  def pr_draft?
- pr_json["mergeable_state"] == "draft"
+ pr_json["draft"] == true
  end
 
  # @!group PR Commit Metadata