Capitalize first letter of PR titles by default in TitleBuilder

Author: kbukum1

common/lib/dependabot/pull_request_creator/message_builder/title_builder.rb

@@ -77,7 +77,7 @@ def prefix
         def capitalize?
           return T.must(prefixer).capitalize_first_word? if prefixer
 
-          false
+          true
         end
 
         # Builds prefix from explicit commit_message_options only.

common/spec/dependabot/pull_request_creator/message_builder/title_builder_spec.rb

@@ -24,8 +24,8 @@
         described_class.new(base_title: "bump lodash from 4.0.0 to 5.0.0")
       end
 
-      it "returns the base title unchanged" do
-        expect(builder.build).to eq("bump lodash from 4.0.0 to 5.0.0")
+      it "capitalizes the first letter by default" do
+        expect(builder.build).to eq("Bump lodash from 4.0.0 to 5.0.0")
       end
     end
 
@@ -52,7 +52,7 @@
       end
 
       it "applies the prefix" do
-        expect(builder.build).to eq("[ci]: bump lodash from 4.0.0 to 5.0.0")
+        expect(builder.build).to eq("[ci]: Bump lodash from 4.0.0 to 5.0.0")
       end
     end
 
@@ -79,7 +79,7 @@
       end
 
       it "does not double-space" do
-        expect(builder.build).to eq("[ci] bump lodash from 4.0.0 to 5.0.0")
+        expect(builder.build).to eq("[ci] Bump lodash from 4.0.0 to 5.0.0")
       end
     end
 
@@ -116,7 +116,7 @@
       end
 
       it "includes scope in the prefix" do
-        expect(builder.build).to eq("chore(deps): bump lodash from 4.0.0 to 5.0.0")
+        expect(builder.build).to eq("chore(deps): Bump lodash from 4.0.0 to 5.0.0")
       end
     end
 
@@ -153,7 +153,7 @@
       end
 
       it "uses the development prefix" do
-        expect(builder.build).to eq("chore: bump eslint from 7.0.0 to 8.0.0")
+        expect(builder.build).to eq("chore: Bump eslint from 7.0.0 to 8.0.0")
       end
     end
 
@@ -198,8 +198,8 @@
         ]
       end
 
-      it "returns the base title without prefix" do
-        expect(builder.build).to eq("bump lodash from 4.0.0 to 5.0.0")
+      it "capitalizes the base title without prefix" do
+        expect(builder.build).to eq("Bump lodash from 4.0.0 to 5.0.0")
       end
     end
   end