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

Simplify calls to render partial #3628

Merged
merged 1 commit into from
Sep 4, 2019
Merged

Simplify calls to render partial #3628

merged 1 commit into from
Sep 4, 2019

Conversation

javierm
Copy link
Member

@javierm javierm commented Jun 22, 2019

Objectives

  • Simplify the code rendering partials
  • Make the code more consistent, since most of our calls to partial rendering omit the partial and locals keys

@javierm javierm self-assigned this Jun 22, 2019
We're also adding a bit of consistency, since most of our calls to
partial rendering omit the `partial` and `locals` keys.
@javierm
Copy link
Member Author

javierm commented Sep 4, 2019

Travis failures are not related to this pull request.

@javierm javierm merged commit 1cff08d into master Sep 4, 2019
@javierm javierm deleted the render_partial branch September 4, 2019 13:53
@javierm javierm added this to Release 1.1.0 in Roadmap Sep 10, 2019
smarques pushed a commit to venetochevogliamo/consul that referenced this pull request Apr 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Roadmap
  
Release 1.1.0
Development

Successfully merging this pull request may close these issues.

None yet

2 participants