@@ -698,18 +698,15 @@ if File.file?(failsafe_report)
698
698
end
699
699
end
700
700
701
- # Don't show some warnings/messages to a bot, it doesn't read them
702
- prFromRealUser = github . pr_author != 'dependabot'
703
-
704
- if prFromRealUser && github . pr_body !~ /Addressed to #\d +/
701
+ if github . pr_body !~ /Addressed to #\d +/
705
702
warn (
706
703
"danger check: pull request description doesn't contain a link to original issue.\n " \
707
704
"Consider adding a comment in the following format: `Addressed to #XXX` where `XXX` is an issue number"
708
705
)
709
706
end
710
707
711
708
commits = git . commits . size
712
- if prFromRealUser && commits > 1
709
+ if commits > 1
713
710
if git . commits . any? { |c | c . message =~ /^Merge branch/ || c . message =~ /^Merge remote-tracking branch/ }
714
711
fail (
715
712
"danger check: pull request contains merge commits! " \
@@ -725,7 +722,7 @@ if prFromRealUser && commits > 1
725
722
end
726
723
end
727
724
728
- if prFromRealUser && github . branch_for_head !~ /^gh[0-9]+_/
725
+ if github . branch_for_head !~ /^gh[0-9]+_/
729
726
warn ( "danger check: branch `#{ github . branch_for_head } ` does not comply with our best practices. " \
730
727
"Branch name should use the following scheme: `ghXXX_meaningful-name` where `XXX` is an issue number. " \
731
728
"**Next time**, please, use this scheme :)"
@@ -741,7 +738,7 @@ if modified_resources && !updated_url
741
738
end
742
739
743
740
all_checks_passed = violation_report [ :errors ] . empty? && violation_report [ :warnings ] . empty? && violation_report [ :messages ] . empty?
744
- if prFromRealUser && all_checks_passed
741
+ if all_checks_passed
745
742
message (
746
743
"@#{ github . pr_author } thank you for the PR! All quality checks have been passed! " \
747
744
"Next step is to wait when @php-coder will review this code"
0 commit comments