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

Missing documentation? #9

Open
kokujin opened this issue Feb 11, 2016 · 9 comments
Open

Missing documentation? #9

kokujin opened this issue Feb 11, 2016 · 9 comments

Comments

@kokujin
Copy link

kokujin commented Feb 11, 2016

  • How does one configure the log messages?
  • How does one change the default action?

I tried this

.pipe( conflict( './', {
                    defaultChoice: 'd'
                } ) )

It failed. Can someone show some examples or point me to the docs that can show me how to deal with these issues? Thanks

@vousk
Copy link

vousk commented Feb 19, 2016

I also can't manage to activate defaultChoice. I'm using it like this:
conflict(my_path, { defaultChoice:'y' })
But it still asks me for manual input...

@vousk
Copy link

vousk commented Feb 19, 2016

Ok I understood, with defaultChoice:'y', the system asks for pressing "enter" to confirm replacement, instead of asking what to do. But it would be usefull to have an other option applyDefaultChoice:'true' to launch the whole process without manual intervention.

@benjaminapetersen
Copy link

I'd like to see a bit more documentation here as well. I'm not 100% sure gulp-conflict does what I think it is intending to do via the README, I could be mistaken. For example:

gulp.src(['./files/*.js'])
  .pipe(conflict('./dir'))
  .pipe(gulp.dest('./dir'));

If I am checking gulp.src(['./files/*.js']) against ./dir via .pipe(conflict('./dir')), then why do I need to .pipe(gulp.dest('./dir')); at the end? It seems like the output is supposed to be conflict/no conflict. Is there conceptually something else going on here?

@benjaminapetersen
Copy link

ping! just checking if the repo is still alive.

@joakimbeng
Copy link
Member

Yes, you're right that there is missing documentation. There are two more options replaceAll and skipAll both of which shouldn't require input from the user.

why do I need to .pipe(gulp.dest('./dir')); at the end?

Because this plugin does not touch the disk, it just filters the stream according to if existing files should be overwritten or not. Without it no write to disk would happen.

@benjaminapetersen
Copy link

So it can diff src files against whatever you do to the stream, but not against another directory of files, correct? Just to clarify, I think then my use case is not possible.

Thanks!

@vousk
Copy link

vousk commented Mar 21, 2016

I've just tried "replaceAll", but instead of replacing only the conflicting files, it re-writes every files. Is it the expected behavior ?
My need is simply to allow the script to answer "replace" for conflicting files, I do not want it to override every files.

@jdnichollsc
Copy link

What happen with the documentation? 👎

@tdmalone
Copy link

Might need to check the source in lieu of documentation at the moment.

I was looking for a way to avoid verbose logging, based on this line it looks like we can define our own logging function to replace the default one. That'd be handy to have in the docs somewhere :)

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

6 participants