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 #2412 -- Register dependencies in post-list shortcode #2417

Merged
merged 8 commits into from Jul 30, 2016

Conversation

Projects
None yet
2 participants
@Kwpolska
Member

Kwpolska commented Jul 30, 2016

This is #2412.

cc @ralsina — requires review due to some refactoring this caused.

Kwpolska added some commits Jul 30, 2016

Pass filename to shortcode functions if requested
Signed-off-by: Chris Warrick <kwpolska@gmail.com>
Fix #2412 -- Register dependencies in post-list shortcode
Signed-off-by: Chris Warrick <kwpolska@gmail.com>
Read .dep files in all compilers
Signed-off-by: Chris Warrick <kwpolska@gmail.com>
Add missing import, fix typo
Signed-off-by: Chris Warrick <kwpolska@gmail.com>
Add site.post_per_input_file (stores source paths)
Signed-off-by: Chris Warrick <kwpolska@gmail.com>
Move depfile support to the Post class
Signed-off-by: Chris Warrick <kwpolska@gmail.com>
Changelog for #2412
Signed-off-by: Chris Warrick <kwpolska@gmail.com>

@Kwpolska Kwpolska added this to the v7.7.12 milestone Jul 30, 2016

Fix tests
Signed-off-by: Chris Warrick <kwpolska@gmail.com>
@ralsina

This comment has been minimized.

Member

ralsina commented Jul 30, 2016

Looks sane to me.

@Kwpolska Kwpolska merged commit 366376a into master Jul 30, 2016

3 of 4 checks passed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@Kwpolska Kwpolska deleted the fix-2412 branch Jul 30, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment