Tim Perry comments

Results 1187 comments of


                                            Tim Perry

Add assert_contains() as a new assertion

No worries! No desperate hurry, just trying to make sure it doesn't stall. Thanks for the testing tools :smile:

Add assert_contains() as a new assertion

@ztombol any thoughts on this?

Add assert_contains() as a new assertion

I haven't used bats for anything any time recently, so I'm not sure I'm afraid! The only clear difference is that you can assert on things other than command output,...

Support for certificatePolicies x509 extension in creation

I was also looking for this, since this is a requirement when generating certificates nowadays, and https://crt.sh/lintcert fails without it. My full code for that extension is a bit simpler...

Screenshotting support

Is there a reason this hasn't been merged yet? This would be really useful to me.

Screenshotting support

Nope, for various reasons (this, and some PhantomJS issues that would need me to get PhantomJS 2 working, which seems to be pretty impractical) I've moved to Karma + Chrome...

git commit will hang if lots of files are in staging, like when merging master

How many files is too many files? How large is the change overall?

git commit will hang if lots of files are in staging, like when merging master

That should still work fine, I've done commits of similar sizes before with no problems. How large are the files overall?

git commit will hang if lots of files are in staging, like when merging master

Is that the size of the diff, or the files themselves? Number of lines would be useful too.

git commit will hang if lots of files are in staging, like when merging master

Hmm, even that should be fine. What git-confirm actually does is: * For each pattern, for each changed file, get the diff: ``` changes=$(git diff -U999999999 -p --cached --color=always --...