Skip to content
This repository was archived by the owner on Dec 11, 2021. It is now read-only.

Build: Added grunt-html #128

Closed
wants to merge 2 commits into from

Conversation

AndSheCodes2
Copy link
Contributor

Fixes gh-126

"grunt-contrib-watch": "0.6.1",
"grunt-csscomb": "3.0.0",
"grunt-git-authors": "2.0.0",
"grunt-html": "1.6.0",
"grunt-html": "^5.0.1",
Copy link
Contributor

Choose a reason for hiding this comment

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

This is interesting, apparently the dependency was already there, but it was unused.

Should remove the ^ to be consistent with other non-ranged dependencies.

@jzaefferer
Copy link
Contributor

This is missing the tasks/options/htmllint.js file. Not sure how the commit ended up without the new file.

@jzaefferer
Copy link
Contributor

@TaMe3971 sorry about the whole rebasing mess, that somehow must ended up with dropping the new file. We can help get that fixed tomorrow.

@sfrisk sfrisk closed this in 4b60d4d Oct 18, 2015
@rohmulan
Copy link
Contributor

@sfrisk @jzaefferer shouldn't we have removed ^ before landing?

@sfrisk
Copy link
Contributor

sfrisk commented Oct 21, 2015

Yeah, we should have, but I hadn't noticed the ^ when I pulled in the PR. I'll submit a PR to fix it.

@AndSheCodes2 AndSheCodes2 deleted the htmllint branch September 22, 2018 16:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants