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

Added 'compiled' signal #2370

Merged
merged 1 commit into from Jun 13, 2016
Merged

Added 'compiled' signal #2370

merged 1 commit into from Jun 13, 2016

Conversation

@tritium21
Copy link
Contributor

@tritium21 tritium21 commented Jun 13, 2016

Adds a signal to nikola.post.Post().compile() after the page is compiled, before anything is done with the compiled file.

addresses #2369

@@ -491,6 +492,12 @@ def wrap_encrypt(path, password):
dest,
self.is_two_file),

signal('compiled').send(dict(

This comment has been minimized.

@Kwpolska

Kwpolska Jun 13, 2016
Member

We like the {} syntax more, I’ll fix that.

@Kwpolska Kwpolska self-assigned this Jun 13, 2016
@Kwpolska Kwpolska added this to the v7.8.0 milestone Jun 13, 2016
@Kwpolska
Copy link
Member

@Kwpolska Kwpolska commented Jun 13, 2016

Looks good, thanks!

(I’ll make some minor style fixes and add this to the changelog.)

@Kwpolska Kwpolska merged commit c71f060 into getnikola:master Jun 13, 2016
1 of 2 checks passed
1 of 2 checks passed
continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Kwpolska added a commit that referenced this pull request Jun 13, 2016
@tritium21 tritium21 deleted the tritium21:post_compile_signal branch Jun 15, 2016
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