Issue publishing pact to broker about pact-js HOT 10 CLOSED

Ah, the pactUrls requires the path to be absolute in the latest version. I'll update the docs accordingly, but for now patching your publish task like this should fix your problem:

var path = require('path');
pactUrls: [path.resolve(process.cwd(), 'pacts')],

from pact-js.

Probably, we'll happily accept a PR if you're willing :)

from pact-js.

Thanks for the opportunity! Very keen to. Hopefully soon!

from pact-js.

that worked perfectly. thanks.

out of interest, @mefellows why is it absolute now instead of relative?

from pact-js.

i will leave this issue opened for now until the doc has been updated.

from pact-js.

Thanks @shavo007, I'll follow up on that, short answer is I'm not sure.

from pact-js.

No worries mate

from pact-js.

Looks like it was an unintended consequence (albeit if you provided it with a directory it needed to be absolute anyway), I'll raise an enhancement request. It seems fairly low priority though.

from pact-js.

👍

from pact-js.

Can there not just be some sensible defaults around this kind of stuff? Like make pactUrls optional and just default it to /pacts under your project root?

from pact-js.