Skip to content
This repository has been archived by the owner on Aug 22, 2021. It is now read-only.

Fixed end of line problem with gulp-html-extract plugin on Windows. A… #143

Merged
merged 1 commit into from Jul 3, 2018

Conversation

sohrabtaee
Copy link
Contributor

@sohrabtaee sohrabtaee commented Jul 3, 2018

Fixed end of line problem with gulp-html-extract plugin on Windows.
Added yarn.lock to gitignore.


This change is Reviewable

@CLAassistant
Copy link

CLAassistant commented Jul 3, 2018

CLA assistant check
All committers have signed the CLA.

@coveralls
Copy link

coveralls commented Jul 3, 2018

Pull Request Test Coverage Report for Build 1080

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 1073: 0.0%
Covered Lines: 6
Relevant Lines: 6

💛 - Coveralls

.gitignore Outdated
@@ -1,4 +1,5 @@
bower_components
node_modules
package-lock.json
coverage
yarn.lock
coverage
Copy link
Contributor

@limonte limonte Jul 3, 2018

Choose a reason for hiding this comment

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

nit:
image

this red symbol means that there's no final newline in this file.

To fix it and prevent such issues in future I recommend this VS Code setting:

  "files.insertFinalNewline": true,

@limonte limonte merged commit 7e253e5 into master Jul 3, 2018
@limonte limonte removed the in review label Jul 3, 2018
@limonte limonte deleted the fix/windows-eol-problem branch July 3, 2018 11:51
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.

None yet

4 participants