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

fix: Replace packpath with pkginfo to get metadata in more reliable f… #175

Merged

Conversation

Thorn1089
Copy link
Contributor

…ashion

Further testing revealed packpath did not work consistently in Jest tests.

…ashion

Further testing revealed packpath did not work consistently in Jest tests.
@Thorn1089
Copy link
Contributor Author

I was not able to get packpath to behave for the Jest tests. Ultimately decided to use a slightly different dependency to retrieve package metadata. It's slightly clunkier requiring an extra file to attach the metadata to but the tests that Travis showed as failed before pass for me now. Sorry for the confusion!

@coveralls
Copy link

coveralls commented Apr 20, 2018

Coverage Status

Coverage decreased (-0.03%) to 94.33% when pulling 94f413b on TomRK1089:feat/message-pact into 801bd7e on pact-foundation:feat/message-pact.

@Thorn1089
Copy link
Contributor Author

@mefellows Again, sorry for the premature merge/tag. Thankfully Travis is smarter than I am. :)

@mefellows mefellows merged commit 5abb32e into pact-foundation:feat/message-pact Apr 20, 2018
@mefellows
Copy link
Member

Thanks @TomRK1089 - no problems. This is a WIP branch so happy to bring things in quicker. I'd be more cautious in master :)

@mefellows
Copy link
Member

FYI v6.0.0-alpha.10 is now live with these changes 🙏

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

Successfully merging this pull request may close these issues.

3 participants