Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Epic] vinyl-fs 2.0 #92

Closed
3 of 6 tasks
phated opened this issue Sep 10, 2015 · 7 comments
Closed
3 of 6 tasks

[Epic] vinyl-fs 2.0 #92

phated opened this issue Sep 10, 2015 · 7 comments

Comments

@phated
Copy link
Member

phated commented Sep 10, 2015

Figured I'd create a 2.0 milestone epic to track what still needs to be done.

  • Remove the watch method from library and vinyl-adapter spec
  • Update vinyl blog post
  • More testing on sourcemap integration
  • Decide on resolution for UTF-8 BOM stripping Redux #83
  • Wrap up/decide on any outstanding issues that we can actually control
  • Update to test on Node 4.0 (remove iojs support)

@contra anything else?

@yocontra
Copy link
Member

@phated nope thats it for going to 2.0, thanks for the post

yocontra pushed a commit that referenced this issue Sep 10, 2015
@iliakan
Copy link

iliakan commented Jan 10, 2016

What's the point of integrating sourcemaps into src/dest? It looks like a completely artificial merging in of a standalone plugin.

Let them stay apart as they should and as they actually do.

@phated
Copy link
Member Author

phated commented Jan 10, 2016

Already done. No point in discussing.

@yocontra
Copy link
Member

@iliakan Reduction of boilerplate, might have been better to do in gulp instead of vinyl-fs - still time to change it, the sourcemap stuff hasn't been put on npm or released formally yet

@iliakan
Copy link

iliakan commented Jan 11, 2016

Gulp actively encourages splitting concerns. Here is no benefit of merging them.
Separate sourcemaps are as concise as an option, but more verbose and configurable.

@phated
Copy link
Member Author

phated commented Jan 11, 2016

I addressed that point in my Thunder Plains Conf talk. Gulp defers stuff to userland religiously but we realize when something is used in almost every task and can make the decision to bring it into core. Sourcemaps is the most notable case. We still depend on the userland module so we get all fixes and features.

@phated
Copy link
Member Author

phated commented Mar 1, 2016

Closing this due to pushing forward on vinyl-fs 3

@phated phated closed this as completed Mar 1, 2016
phated pushed a commit that referenced this issue Nov 27, 2017
phated pushed a commit that referenced this issue Nov 27, 2017
updated as part of #92, removing iojs and adding in node.js 4.0, 4.1
phated pushed a commit that referenced this issue Nov 27, 2017
phated pushed a commit that referenced this issue Nov 27, 2017
updated as part of #92, removing iojs and adding in node.js 4.0, 4.1
phated pushed a commit that referenced this issue Nov 27, 2017
phated pushed a commit that referenced this issue Nov 27, 2017
updated as part of #92, removing iojs and adding in node.js 4.0, 4.1
phated pushed a commit that referenced this issue Nov 27, 2017
phated pushed a commit that referenced this issue Nov 27, 2017
updated as part of #92, removing iojs and adding in node.js 4.0, 4.1
phated pushed a commit that referenced this issue Nov 27, 2017
phated pushed a commit that referenced this issue Nov 27, 2017
updated as part of #92, removing iojs and adding in node.js 4.0, 4.1
phated pushed a commit that referenced this issue Nov 27, 2017
phated pushed a commit that referenced this issue Nov 27, 2017
updated as part of #92, removing iojs and adding in node.js 4.0, 4.1
phated pushed a commit that referenced this issue Nov 27, 2017
phated pushed a commit that referenced this issue Nov 27, 2017
updated as part of #92, removing iojs and adding in node.js 4.0, 4.1
phated pushed a commit that referenced this issue Nov 28, 2017
phated pushed a commit that referenced this issue Nov 28, 2017
updated as part of #92, removing iojs and adding in node.js 4.0, 4.1
phated pushed a commit that referenced this issue Nov 28, 2017
phated pushed a commit that referenced this issue Nov 28, 2017
updated as part of #92, removing iojs and adding in node.js 4.0, 4.1
phated pushed a commit that referenced this issue Nov 28, 2017
phated pushed a commit that referenced this issue Nov 28, 2017
updated as part of #92, removing iojs and adding in node.js 4.0, 4.1
phated pushed a commit that referenced this issue Nov 28, 2017
phated pushed a commit that referenced this issue Nov 28, 2017
updated as part of #92, removing iojs and adding in node.js 4.0, 4.1
phated pushed a commit that referenced this issue Nov 28, 2017
phated pushed a commit that referenced this issue Nov 28, 2017
updated as part of #92, removing iojs and adding in node.js 4.0, 4.1
phated pushed a commit that referenced this issue Nov 28, 2017
phated pushed a commit that referenced this issue Nov 28, 2017
updated as part of #92, removing iojs and adding in node.js 4.0, 4.1
phated pushed a commit that referenced this issue Nov 28, 2017
phated pushed a commit that referenced this issue Nov 28, 2017
updated as part of #92, removing iojs and adding in node.js 4.0, 4.1
phated pushed a commit that referenced this issue Nov 28, 2017
phated pushed a commit that referenced this issue Nov 28, 2017
updated as part of #92, removing iojs and adding in node.js 4.0, 4.1
phated pushed a commit that referenced this issue Nov 28, 2017
phated pushed a commit that referenced this issue Nov 28, 2017
updated as part of #92, removing iojs and adding in node.js 4.0, 4.1
phated pushed a commit that referenced this issue Nov 28, 2017
phated pushed a commit that referenced this issue Nov 28, 2017
updated as part of #92, removing iojs and adding in node.js 4.0, 4.1
phated pushed a commit that referenced this issue Nov 28, 2017
phated pushed a commit that referenced this issue Nov 28, 2017
updated as part of #92, removing iojs and adding in node.js 4.0, 4.1
phated pushed a commit that referenced this issue Nov 28, 2017
phated pushed a commit that referenced this issue Nov 28, 2017
updated as part of #92, removing iojs and adding in node.js 4.0, 4.1
phated pushed a commit that referenced this issue Nov 28, 2017
phated pushed a commit that referenced this issue Nov 28, 2017
updated as part of #92, removing iojs and adding in node.js 4.0, 4.1
phated pushed a commit that referenced this issue Nov 28, 2017
updated as part of #92, removing iojs and adding in node.js 4.0, 4.1
phated pushed a commit that referenced this issue Dec 5, 2017
phated pushed a commit that referenced this issue Dec 5, 2017
updated as part of #92, removing iojs and adding in node.js 4.0, 4.1
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

No branches or pull requests

3 participants