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

Standardize how tilde is used for templating in core plans #986

Conversation

predominant
Copy link
Collaborator

A re-implementation of #853.

Signed-off-by: Graham Weldon [email protected]

@thesentinels
Copy link
Contributor

Thanks for the pull request! Here is what will happen next:

  1. Your PR will be reviewed by the maintainers
  2. If everything looks good, one of them will approve it, and your PR will be merged.

Thank you for contributing!

@predominant predominant requested review from fnichol, reset and a team as code owners November 19, 2017 09:13
Copy link
Contributor

@tduffield tduffield left a comment

Choose a reason for hiding this comment

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

@predominant looks like Travis is failing :(

@predominant
Copy link
Collaborator Author

@tduffield Yeah, its failing on stuff that I don't know how to work around right now, where inclusions are unable to be sourced and checked due to templating:

Unable to check included file: {{pkg.svc_config_path}}/environment.sh
Unable to check included file: {{pkg.svc_config_path}}/functions.sh
Unable to check included file: {{pkg.svc_config_path}}/functions.sh
Unable to check included file: {{pkg.svc_config_path}}/environment.sh
Unable to check included file: {{pkg.svc_config_path}}/functions.sh
Unable to check included file: {{pkg.svc_config_path}}/environment.sh
Unable to check included file: {{pkg.svc_config_path}}/functions.sh

@nellshamrell
Copy link
Contributor

Unfortunately, do to the age of this PR and the number of plans it changes, along with the merge conflicts that have emerged due to the age, I'm going to go ahead and close this PR. @predominant - if this is something you would still like to see done, please feel free to open an RFC, we require an RFC for any pull request that is going to affect more than 1 or 2 plans. Thank you for the work you put into this, it is still very much appreciated!

@predominant
Copy link
Collaborator Author

Thanks @nellshamrell

@predominant predominant deleted the predominant/template-standards-reboot branch April 5, 2018 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants