{"action":"created","issue":{"url":"https://api.github.com/repos/DanielCaspers/gulp-flat/issues/1","repository_url":"https://api.github.com/repos/DanielCaspers/gulp-flat","labels_url":"https://api.github.com/repos/DanielCaspers/gulp-flat/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/DanielCaspers/gulp-flat/issues/1/comments","events_url":"https://api.github.com/repos/DanielCaspers/gulp-flat/issues/1/events","html_url":"https://github.com/DanielCaspers/gulp-flat/issues/1","id":135620240,"number":1,"title":"Fork of working version","user":{"login":"akim-mcmath","id":15257150,"avatar_url":"https://avatars.githubusercontent.com/u/15257150?v=3","gravatar_id":"","url":"https://api.github.com/users/akim-mcmath","html_url":"https://github.com/akim-mcmath","followers_url":"https://api.github.com/users/akim-mcmath/followers","following_url":"https://api.github.com/users/akim-mcmath/following{/other_user}","gists_url":"https://api.github.com/users/akim-mcmath/gists{/gist_id}","starred_url":"https://api.github.com/users/akim-mcmath/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/akim-mcmath/subscriptions","organizations_url":"https://api.github.com/users/akim-mcmath/orgs","repos_url":"https://api.github.com/users/akim-mcmath/repos","events_url":"https://api.github.com/users/akim-mcmath/events{/privacy}","received_events_url":"https://api.github.com/users/akim-mcmath/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-02-23T03:32:20Z","updated_at":"2016-02-23T05:58:53Z","closed_at":null,"body":"Hi,\r\n\r\nSince you [invited contributions](http://stackoverflow.com/questions/35564976/nodejs-gulp-streams-vinyl-file-objects-gulp-wrapper-for-npm-package-produci), I decided I would give it a shot. You can view my fork of the repo [here](https://github.com/akim-mcmath/gulp-flat). Feel free to look it over, clone it, or whatever you like. If you would like, I can submit a pull request. If it's not to your liking, don't worry about it. But I thought I would try an implementation of the project, and hopefully it will at least give you some ideas.\r\n\r\nHere are the changes that I made:\r\n\r\n* Fixed the issue with buffered mode, so that the output displays correctly.\r\n* Added support for streaming mode.\r\n* Omitted [through2](https://github.com/rvagg/through2) as a dependency, since it lacks support for\r\nutilizing the [Transform Stream's](https://nodejs.org/api/stream.html#stream_class_stream_transform_1) contstructor, which was necessary to implement\r\nthe streaming mode version.\r\n* Added tests using [mocha](https://mochajs.org/) and [chai](http://chaijs.com/api/).\r\n* Added coverage support using [Istanbul](https://github.com/gotwarlost/istanbul).\r\n* Added linting support using [JSHint](http://jshint.com/about/).\r\n* Added configuration files: [.editorconfig](http://editorconfig.org/), .gitignore, .jshintrc.\r\n* Prepared repo for continuous intigration with [Travis-CI](https://travis-ci.org/) by including a .travis.yml config file\r\n*Prepared repo for coverage reporting with [Coveralls](https://coveralls.io/), if you would like\r\nto display the relevant badges.\r\n* Updated README with API section.\r\n* Updated package.json with scripts, dependencies, and other relevant\r\n   fields.\r\n\r\nThe following tasks still need to be done:\r\n\r\n* If you want to display CI and coverage badges, you should\r\nadd this repo to your Travis-CI and Coveralls accounts. The repo is\r\nalready configured for those services.\r\n* You should also update the README with installation instructions, perhaps\r\nan example or two, and any badges you want to display."},"comment":{"url":"https://api.github.com/repos/DanielCaspers/gulp-flat/issues/comments/187552219","html_url":"https://github.com/DanielCaspers/gulp-flat/issues/1#issuecomment-187552219","issue_url":"https://api.github.com/repos/DanielCaspers/gulp-flat/issues/1","id":187552219,"user":{"login":"DanielCaspers","id":6056563,"avatar_url":"https://avatars.githubusercontent.com/u/6056563?v=3","gravatar_id":"","url":"https://api.github.com/users/DanielCaspers","html_url":"https://github.com/DanielCaspers","followers_url":"https://api.github.com/users/DanielCaspers/followers","following_url":"https://api.github.com/users/DanielCaspers/following{/other_user}","gists_url":"https://api.github.com/users/DanielCaspers/gists{/gist_id}","starred_url":"https://api.github.com/users/DanielCaspers/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DanielCaspers/subscriptions","organizations_url":"https://api.github.com/users/DanielCaspers/orgs","repos_url":"https://api.github.com/users/DanielCaspers/repos","events_url":"https://api.github.com/users/DanielCaspers/events{/privacy}","received_events_url":"https://api.github.com/users/DanielCaspers/received_events","type":"User","site_admin":false},"created_at":"2016-02-23T05:58:52Z","updated_at":"2016-02-23T05:58:52Z","body":"Hi Akim,\n\nI will definitely be looking over this more closely tomorrow (which is why\nI haven't marked the post as resolved yet). I think the fact that you went\nthis far is just stellar! I really appreciate that you helped this along.\nIn fact, my original contributions seem somewhat insignificant in\ncomparison now in my opinion, but it at least it was a learning experience\ntoo.\n\nAlso, if I decide to go the pull request route, what are your thoughts on\nme rebasing some of this and expanding the commits? My initial thought is\nthat if I don't want to pull in all of the changes (for example, if some of\nthe tools violate the flat license (which I am actually currently in\nviolation of and must change), would you be ok with the work being\nseparated out so that the majority gets in, but some unique features stay\nin your repo?\n\nWhat are your thoughts on co-authoring this on NPM once we both deem it\nworthy of posting? I would be more than happy to co-author this with you if\nyou would accept!\n\nSincerely,\nDaniel Caspers\n\n\nOn Mon, Feb 22, 2016 at 9:32 PM, Akim McMath <notifications@github.com>\nwrote:\n\n> Hi,\n>\n> Since you invited contributions\n> <http://stackoverflow.com/questions/35564976/nodejs-gulp-streams-vinyl-file-objects-gulp-wrapper-for-npm-package-produci>,\n> I decided I would give it a shot. You can view my fork of the repo here\n> <https://github.com/akim-mcmath/gulp-flat>. Feel free to look it over,\n> clone it, or whatever you like. If you would like, I can submit a pull\n> request. If it's not to your liking, don't worry about it. But I thought I\n> would try an implementation of the project, and hopefully it will at least\n> give you some ideas.\n>\n> Here are the changes that I made:\n>\n>    - Fixed the issue with buffered mode, so that the output displays\n>    correctly.\n>    - Added support for streaming mode.\n>    - Omitted through2 <https://github.com/rvagg/through2> as a\n>    dependency, since it lacks support for utilizing the Transform Stream's\n>    <https://nodejs.org/api/stream.html#stream_class_stream_transform_1>\n>    contstructor, which was necessary to implement the streaming mode version.\n>    - Added tests using mocha <https://mochajs.org/> and chai\n>    <http://chaijs.com/api/>.\n>    - Added coverage support using Istanbul\n>    <https://github.com/gotwarlost/istanbul>.\n>    - Added linting support using JSHint <http://jshint.com/about/>.\n>    - Added configuration files: .editorconfig <http://editorconfig.org/>,\n>    .gitignore, .jshintrc.\n>    - Prepared repo for continuous intigration with Travis-CI\n>    <https://travis-ci.org/> by including a .travis.yml config file\n>    *Prepared repo for coverage reporting with Coveralls\n>    <https://coveralls.io/>, if you would like to display the relevant\n>    badges.\n>    - Updated README with API section.\n>    - Updated package.json with scripts, dependencies, and other relevant\n>    fields.\n>\n> The following tasks still need to be done:\n>\n>    - If you want to display CI and coverage badges, you should add this\n>    repo to your Travis-CI and Coveralls accounts. The repo is already\n>    configured for those services.\n>    - You should also update the README with installation instructions,\n>    perhaps an example or two, and any badges you want to display.\n>\n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/DanielCaspers/gulp-flat/issues/1>.\n>\n\n\n\n-- \nDan Caspers\nKeyboardist for LikeABossa <http://likeabossa.com/>\n952-270-6622 <+19522706622> | LinkedIn Profile\n<https://www.linkedin.com/in/danielcaspers>\n"}}