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

Revert "Revert "Inline the render_ds_profile_header"" #2168

Merged
merged 2 commits into from
Jul 6, 2020
Merged

Conversation

jcoyne
Copy link
Collaborator

@jcoyne jcoyne commented Jul 6, 2020

Why was this change made?

Re-add the changes that were previously added in #2157 and reverted in #2165. Now with tests and fixes.

How was this change tested?

Test suite and tested on stage.

Which documentation and/or configurations were updated?

n/a

@jcoyne jcoyne changed the title Revert "Revert "Inline the render_ds_profile_header"" [HOLD] Revert "Revert "Inline the render_ds_profile_header"" Jul 6, 2020
@jcoyne jcoyne force-pushed the revert-revert branch 2 times, most recently from e64e15f to 86e80bb Compare July 6, 2020 16:56
@jcoyne jcoyne changed the title [HOLD] Revert "Revert "Inline the render_ds_profile_header"" Revert "Revert "Inline the render_ds_profile_header"" Jul 6, 2020
And fix problems the test finds
Copy link
Member

@mjgiarlo mjgiarlo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adding the tests!

@jcoyne jcoyne merged commit c1165e4 into master Jul 6, 2020
@jcoyne jcoyne deleted the revert-revert branch July 6, 2020 17:56
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