-
Notifications
You must be signed in to change notification settings - Fork 1
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
[Ready] Migrate functionality from Publisher #5
Conversation
bad0d54
to
98785cf
Compare
@Integralist @darnould @stevenjack Just committed the last spec, so now ready for review. Note that some of the unit tests are missing (raised issue #6). 🏄 |
Will need to bump the version up from |
'components', | ||
'foo' | ||
) | ||
subject { Alephant::Renderer } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm pretty sure that subject
is automatically set to an instance of Alephant::Renderer
implicitly as it's referenced in the describe above.
The CI job was failing due to hard-coding the Rspec options into each Rake task - as To fix this I used rake-rspec and rely upon users having a .rspec file in their home directory. |
Problem
Currently the rendering functionality of Alephant sits within the Publisher (moved in #5). Due to a new publisher gem being created to deal with messages that are received via a request, the functionality needs to be extracted back out into this gem. Note: this gem is not currently being used anywhere.
Solution
Move the current rendering functionality into this gem, ignoring what is currently in this repo. After this is done the Publisher will be refactored to use this gem.