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

Print kramdown warnings #519

Merged
merged 1 commit into from Jan 11, 2015
Merged

Print kramdown warnings #519

merged 1 commit into from Jan 11, 2015

Conversation

@ddfreyne
Copy link
Member

@ddfreyne ddfreyne commented Jan 11, 2015

Potential fix for #459.

The warnings are not particularly useful, since they do not contain line number information; that information is unfortunately not provided by kramdown yet.

@ddfreyne
Copy link
Member Author

@ddfreyne ddfreyne commented Jan 11, 2015

CC @nanoc/contributors — +1s please

Loading

@mpapis
Copy link
Member

@mpapis mpapis commented Jan 11, 2015

👍 the failing tests look like mustache issue

Loading

ddfreyne added a commit that referenced this issue Jan 11, 2015
@ddfreyne ddfreyne merged commit 6c69796 into release-3.7.x Jan 11, 2015
1 check failed
Loading
@ddfreyne ddfreyne deleted the print-kramdown-warnings branch Jan 11, 2015
ddfreyne added a commit that referenced this issue Jan 11, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants