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

This will allow plugins to load in prioritized sequence #511

Merged
merged 1 commit into from
May 25, 2013

Conversation

neilbaylorrulez
Copy link
Contributor

Hey there

This fix closes issue #502.

Here's the change:
neilbaylorrulez@022d01a

Please let me know if you have any questions :)

balupton added a commit that referenced this pull request May 25, 2013
- v6.35.0 May 25, 2013
	- We now respect plugin priorities again
		- Thanks to [Neil Taylor](https://github.com/neilbaylorrulez) for
[pull request #511](#511)
		- Set plugin priorities by `priority: 500` or whatever in your plugin
class
		- Set event specific priorities by `eventNamePriority: 500` or
whatever in your plugin class
	- Updated dependencies
@balupton balupton merged commit 022d01a into docpad:master May 25, 2013
@balupton
Copy link
Member

Thanks for that. Merged it in but the tests failed. Appeared that it was causing everything to re-render twice for some reason. I've fixed it by moving the priority functionality into bal-util's EventEmitterEnhanced directly: bevry-archive/bal-util@9bd0ef2

You can now also set event specific priorities by doing eventNamePriority: 500

Released to v6.35.0

balupton added a commit that referenced this pull request Oct 23, 2013
- v6.35.0 May 25, 2013
	- We now respect plugin priorities again
		- Thanks to [Neil Taylor](https://github.com/neilbaylorrulez) for
[pull request #511](#511)
		- Set plugin priorities by `priority: 500` or whatever in your plugin
class
		- Set event specific priorities by `eventNamePriority: 500` or
whatever in your plugin class
	- Updated dependencies
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.

2 participants