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

Restore previous repo behavior of remark-validate-links #32

Merged
merged 1 commit into from
Aug 9, 2019

Conversation

vweevers
Copy link
Owner

@vweevers vweevers commented Aug 9, 2019

Since 9.0.0, remark-validate-links tries to get the repo url from git remotes (which is not desirable for forks) rather than the package.json. Hallmark now reads package.json itself and passes the repo url to remark-validate-links.

I also inlined some code for finding & reading the package.json so that this doesn't have to be done 3 times over (by pkg-config, deglob and for getting the repo url).

Since 9.0.0, remark-validate-links tries to get the repo url from
git remotes (which is not desirable for forks) rather than the
package.json. Hallmark now reads package.json itself and passes
the repo url to remark-validate-links.
@vweevers vweevers added the semver-patch Bug fixes that are backward compatible label Aug 9, 2019
vweevers added a commit to vweevers/node-docker-machine that referenced this pull request Aug 9, 2019
@vweevers vweevers merged commit 48c7abc into master Aug 9, 2019
@vweevers vweevers deleted the restore-old-validate-links-behavior branch August 9, 2019 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver-patch Bug fixes that are backward compatible
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant