Skip to content

Update dependencies and fix test#128

Open
nulltask wants to merge 7 commits intokarlfreeman:masterfrom
nulltask:feature/fix-test
Open

Update dependencies and fix test#128
nulltask wants to merge 7 commits intokarlfreeman:masterfrom
nulltask:feature/fix-test

Conversation

@nulltask
Copy link
Contributor

@nulltask nulltask commented Jan 10, 2017

Bump version to middleman >= 4.1.11 and updated some dependencies to pass tests. Also I updated travis.yml. Middleman 4.1.11+ is no longer support Ruby < 2.2.2

@nulltask nulltask changed the title Update dependency and fix test Update dependencies and fix test Jan 10, 2017
@nulltask
Copy link
Contributor Author

@karlfreeman I fixed to pass all test. Please review :) /cc @middleman-contrib/people

Cucumber::Rake::Task.new(:cucumber, 'Run features that should pass') do |t|
exempt_tags = ['--tags ~@wip']
t.cucumber_opts = "--color #{exempt_tags.join(' ')} --strict --format #{ENV['CUCUMBER_FORMAT'] || 'Fivemat'}"
t.cucumber_opts = "--color #{exempt_tags.join(' ')} --strict # --format #{ENV['CUCUMBER_FORMAT'] || 'Fivemat'}"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there any reason why the formatter option is commented out?

require 'middleman-core'
require 'middleman-core/step_definitions'

require 'codeclimate-test-reporter'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this project still using Codeclimate?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants