Skip to content
This repository has been archived by the owner on Feb 1, 2022. It is now read-only.

Warns about pull-* classes in media #153

Closed
wants to merge 2 commits into from

Conversation

iamphill
Copy link
Contributor

With v3.3, pull-left & pull-right were deprecated inside media

This warns rather than error. I guess error it when support is dropped?

New warning number W010

@iamphill
Copy link
Contributor Author

The travis fail looks to be because of the Bootstrap docs using pull-left & pull-right.

@@ -721,6 +721,12 @@ var semver = require('semver');
reporter('Neither .form-inline nor .form-horizontal should be used directly on a `.form-group`. Instead, nest the .form-group within the .form-inline or .form-horizontal', badFormGroups);
}
});
addLinter("W010", function lintMediaPulls($, reporter) {
var mediaPulls = $('.media').children('.pull-left, .pull-right');
Copy link
Collaborator

Choose a reason for hiding this comment

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

Use $('.media>.pull-left, .media>.pull-right')

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling e2216a5 on iamphill:media-pulls into eb17c13 on twbs:master.

@cvrebert cvrebert closed this in 55bcdb8 Oct 30, 2014
iamphill added a commit to iamphill/bootlint that referenced this pull request Oct 31, 2014
iamphill added a commit to iamphill/bootlint that referenced this pull request Oct 31, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants