search

crissdev / gulp-yaml

A Gulp plugin to convert YAML to JSON
MIT License
24 stars 7 forks source link

Feature/major update #8

Closed ehmicky closed 6 years ago

ehmicky commented 6 years ago

This is a proposal for a major update of gulp-yaml. It includes both changes from the refactor branch and the two currently opened pull requests: #3 and #7.

User features

Add options.ext to specify the filename extension (defaults to .json).

Add options.json and options.onWarning from js-yaml.

Add support for file.contents being null.

Documentation

Improve README.md.

Dependencies

Remove gulp-util deprecated dependency.

Upgrade js-yaml from 3.4.3 to 3.11.0

Upgrade bufferstreams from 1.1.0 to 2.0.0

Add package-lock.json.

Linting

Add linting with ESLint. It is based on Standard JavaScript with two rules exceptions to keep the current coding style untouched.

Refactoring

Major code refactoring.

Note

I see there has been only one commit since November 2015. If you need someone to maintain this project, I can volunteer!

crissdev commented 6 years ago

I see there has been only one commit since November 2015. If you need someone to maintain this project, I can volunteer!

Your help is welcome. I've added you as a collaborator 😄 Thanks.

ehmicky commented 6 years ago

Alright fixed everywhere. Waiting for your green light.

About being added as a collaborator: great! I'll probably submit new PRs in coming weeks.

ehmicky commented 6 years ago

Also just posted this: https://github.com/nodeca/js-yaml/issues/415 And this: https://github.com/yaml/www.yaml.org/pull/25