Rakefile in facter-4.0.18 vs Rakefile in facter-4.0.19
- old
+ new
@@ -6,39 +6,9 @@
RSpec::Core::RakeTask.new(:spec)
Dir.glob(File.join('tasks/**/*.rake')).each { |file| load file }
task default: :spec
-desc 'verify that commit messages match CONTRIBUTING.md requirements'
-task(:commits) do
- # This rake task looks at the summary from every commit from this branch not
- # in the branch targeted for a PR. This is accomplished by using the
- # TRAVIS_COMMIT_RANGE environment variable, which is present in travis CI and
- # populated with the range of commits the PR contains. If not available, this
- # falls back to `master..HEAD` as a next best bet as `master` is unlikely to
- # ever be absent.
- commit_range = ENV['TRAVIS_COMMIT_RANGE'].nil? ? 'master..HEAD' : ENV['TRAVIS_COMMIT_RANGE'].sub(/\.\.\./, '..')
- puts "Checking commits #{commit_range}"
- `git log --no-merges --pretty=%s #{commit_range}`.each_line do |commit_summary|
- # This regex tests for the currently supported commit summary tokens: maint, doc, gem, or fact-<number>.
- # The exception tries to explain it in more full.
- if /^\((maint|doc|docs|gem|fact-\d+)\)|revert/i.match(commit_summary).nil?
- raise "\n\n\n\tThis commit summary didn't match CONTRIBUTING.md guidelines:\n" \
- "\n\t\t#{commit_summary}\n" \
- "\tThe commit summary (i.e. the first line of the commit message) should start with one of:\n" \
- "\t\t(FACT-<digits>) # this is most common and should be a ticket at tickets.puppet.com\n" \
- "\t\t(docs)\n" \
- "\t\t(docs)(DOCUMENT-<digits>)\n" \
- "\t\t(maint)\n" \
- "\t\t(gem)\n" \
- "\n\tThis test for the commit summary is case-insensitive.\n\n\n"
- else
- puts commit_summary.to_s
- end
- puts '...passed'
- end
-end
-
def retrieve_from_keyboard
return unless ARGV =~ /changelog/
puts "Please provide the next release tag:\n"
next_version = $stdin.gets.chomp