Skip to content

Option to allow single tag (no end tag)#279

Open
specious wants to merge 4 commits intoklei:masterfrom
specious:feat/single-tag
Open

Option to allow single tag (no end tag)#279
specious wants to merge 4 commits intoklei:masterfrom
specious:feat/single-tag

Conversation

@specious
Copy link

@joakimbeng, I'd like to extend to you my appreciation for writing this gulp plug-in.

I've got a project that uses gulp-inject, which injects CSS and JS contents into the HTML. I realized it would be nice to be able to omit the closing tag, to just have placeholders like this:

<!DOCTYPE html>
<html>
<head>
  <title>Single file application</title>
  <!-- inject:css -->
</head>
<body>
  <!-- inject:js -->
</body>
</html>

This pull requests adds an option named singleTag which, if set to true, will no longer require a matching end tag.

If you would kindly take a look at this and consider merging it, I would love to switch to using a newer published version of gulp-inject with the ability to do this.

Also, if you don't mind, I might also like to contribute some refactoring improvements which could make it easier for people to make further contributions to this project.

@specious
Copy link
Author

This branch can be used right way with package.json:

{
  "devDependencies": {
    "gulp-inject": "klei/gulp-inject#741048f"
  }
}

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.

1 participant