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
Extract ActionView from Rails #762
Conversation
6928012
to
75e9220
Compare
2224b72
to
af16bae
Compare
75e9220
to
1befbc6
Compare
af16bae
to
3769229
Compare
12c95ff
to
3ca366a
Compare
e4a8967
to
2c3d927
Compare
3ca366a
to
2d893f9
Compare
Reworked this a little bit:
Might want to consider adding some tests for using ActionView in a standalone way in the future, but for now the existing Rails tests should be sufficient. Should be ready for review @brettlangdon |
2c3d927
to
df6ad3d
Compare
2d893f9
to
6dcbe4c
Compare
df6ad3d
to
9414b9f
Compare
7d270cb
to
3ec2bde
Compare
9414b9f
to
9bd69e7
Compare
4691e6a
to
29a1054
Compare
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.
Still have a few open questions, but otherwise looks fine
29a1054
to
322086b
Compare
As a next step towards simplifying the Rails integration, this pull request extracts all the ActionView instrumentation from the Rails integration into a new
action_view
integration. This new integration is automatically activated by the Rails integration, so users withuse :rails
should see no change.Changes in this PR are dependent on #747. After that's merged, rebase and merge this to the dev branch.