ChrisWren / grunt-link-checker

Run node-simple-crawler to discover broken links on your website
MIT License
33 stars 9 forks source link

Rename task to `linkChecker` #21

Closed XhmikosR closed 9 years ago

XhmikosR commented 9 years ago

This should make things simpler to work with grunt templates.

    connect: {
      server: {
        options: {
          port: '<% linkChecker.options.initialPort %>',
          base: 'test/fixtures'
        }
      }
    }

A major version bump should be made though.

ChrisWren commented 9 years ago

Is this a common convention to use camelCase for task names?

XhmikosR commented 9 years ago

Generally, it's a convention to use camelcase everywhere.

Here is a branch with this change: https://github.com/ChrisWren/grunt-link-checker/compare/master...XhmikosR:rename-task

ChrisWren commented 9 years ago

Sounds good to me. If you can make a branch with just that change I am willing to merge it.

XhmikosR commented 9 years ago

Done, see #23.